Re: [SSSD] [PATHC] 1625-Confusing error messages for invalid sssd.conf

[Ariel Barria]

Thanks for your comments.
> Date: Wed, 28 Nov 2012 14:38:29 +0100
> From: [e-mail]
> To: [e-mail]
> Subject: Re: [SSSD] [PATHC] 1625-Confusing error messages for invalid	sssd.conf
>
> On 28/11/12 01:31, Ariel Barria wrote:
> > https://fedorahosted.org/sssd/ticket/1625
> >
> > /
> > //    Additional info:/
> >    ..../the class of configuration error messages could be
> >    changed to match the default debug level./
> >
> > Apparently this was covered with #1345 and in server.c
> >
> > /  if (debug_level == SSSDBG_UNRESOLVED) {
> >          /* set debug level if any in conf_entry */
> >          ret = confdb_get_int(ctx->confdb_ctx, conf_entry,
> >                               CONFDB_SERVICE_DEBUG_LEVEL,
> >                               SSSDBG_DEFAULT,
> >                               &debug_level);/
>
> Hi Ariel,
>
> Could you add the path of database to the
> > DEBUG(SSSDBG_CRIT_FAILURE, ("Error loading configuration file: "
> >                             "[%d]: %s\n", ret, strerror(ret)));
>
> part as well? And error messages from strerror are not much help in this 
> case
> [22]: Invalid argument // for empty file
> [2]: No such file or directory // for invalid syntax in config file
> so could you change this accordingly to ticket proposal ?
>
> Thanks
>
> Ondra
> > sssd-devel mailing list
> > [e-mail]
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
> -- 
> Ondrej Kos
> Associate Software Engineer
> Identity Management
> Red Hat Czech
>
> phone: [phone]
> cell:  [phone]
> ext:   82-62558
> loc:   1013 Brno 1 office
> irc:   okos @ #brno
> _______________________________________________
> sssd-devel mailing list
> [e-mail]
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel