On 09/29/2014 05:58 PM, Michal Židek wrote:
Hi,

this patch solves the ticket:
https://fedorahosted.org/sssd/ticket/2361

From the discussion on bugzilla and the ticket comments, it
looks like Jakub and Lukas were for changes in documentation
only (+ maybe warning/error if user misconfigured SSSD).

I do not want look like I ignore the opinions of others
but I think the best way is to simply change the defaults
to the same as expected by AD backend. While I do agree
that users should set their id_provider option to AD
if they use AD backend, this option is more or less
AD specific, so I think it would be better to have
the same defaults with id_provider ldap. This
would also solve the small regressions that are
mentioned in the bugzilla (by users who do use
id_provider=AD and id_mapping=true).

OTOH if this approach is disliked by other developers
I have no big problems with the man page + warning/error
solution.

Simple patch is attached.

Thanks,
Michal


ACK,

this patch is also useful for https://fedorahosted.org/sssd/ticket/2117
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel