On Tue, Jul 23, 2013 at 05:11:55PM +0200, Michal Židek wrote:
On 07/23/2013 02:29 PM, Jakub Hrozek wrote:
>The subdomain users user FQDN in their name attribute. However, handling
>of whether to use FQDN in the LDAP code was not really good. This patch
>introduces a utility function and converts code that was relying on
>user/group names matching to this utility function.
>
>This is a temporary fix until we can refactor the sysdb API in #2011.
>
>Alexander, this patch fixed the username issues for me. I still see one
>more problem -- on the first ID lookup, the user is reported as a member
>of "Domain Users", but not on the subsequent lookups. This is unrelated
>problem I'm looking into, but the attached patch is still correct.
>
>
Nack.
The function sdap_get_primary_name() is written for groups only.
Variable names and debug messages are all group specific. Also
the SDAP_AT_GROUP_NAME is used unconditionally inside of this function.
It works, but only because SDAP_AT_GROUP_NAME and SDAP_AT_USER_NAME have
the same value.
I did some basic testing and the functionality looks good. I'll do more
testing with the final patch where the above mentioned is fixed.
Thanks
Michal
You're completely right, the function only worked by accident. Thanks.
New patch is attached.