Re: [SSSD] [PATCH] If the same server is in both primary and secondary server, list, we must only add it to the primary list.

[Pavel Březina]

On 08/09/2012 10:50 AM, Michal Zidek wrote:
> This patch solves 2 things mentioned in ticket 1463.
> a) If the same server is in both primary and secondary server list, we
> must only add it to the primary list.
>
> b) SRV resolution for backup servers should not be permitted. It now
> prints error message if the sssd.conf
> is configured that way.
>
> https://fedorahosted.org/sssd/ticket/1463
>
> The patch is attached.
>
> Thanks
> Michal

Nack.
Duplicates are already handled in be_fo_add_server() and 
be_fo_add_srv_server() so it does not require any changes. But you can 
add a debug message that will inform about adding a duplicate.

The srv part looks good.