summaryrefslogtreecommitdiffstats
path: root/src/providers/proxy
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2015-05-22 15:19:31 +0200
committerJakub Hrozek <jhrozek@redhat.com>2015-05-31 19:27:45 +0200
commitee44aac95e42c3cb634876286a2aa4960ac69a2b (patch)
tree7f6d2a4591acb372d153906454d4564807048247 /src/providers/proxy
parentfd3b0d8235322ada8f3b9b83b30ce57242ebf6cd (diff)
downloadsssd-ee44aac95e42c3cb634876286a2aa4960ac69a2b.tar.gz
sssd-ee44aac95e42c3cb634876286a2aa4960ac69a2b.tar.xz
sssd-ee44aac95e42c3cb634876286a2aa4960ac69a2b.zip
Download complete groups if ignore_group_members is set with tokengroups
Resolves: https://fedorahosted.org/sssd/ticket/2644 When tokenGroups are enabled, we save groups using their SID as the RDN attribute during initgroups() and later, if the groups is requested and saved again with the full name, remove the original and save the new group entry. Saving the new group entry would break if ignore_group_members is also set, because the new group entry would lack the "member" attribute, so the member/memberof links between the new group and the user entry wouldn't be established again. This patch changes the initgroups processing so that the full group object is fetched when initgroups is enabled but together with ignore_group_members. This solution imposes some performance impact, because instead of one search for tokenGroups we also need to resolve the groups. The more systematic solution would be to get rid of removing the group entry as described in https://fedorahosted.org/sssd/ticket/2656 To reproduce the bug, set: ignore_group_members = True with a backend that uses: id_provider = ad Then run: $ id aduser@ad_domain.com $ id aduser@ad_domain.com Reviewed-by: Sumit Bose <sbose@redhat.com>
Diffstat (limited to 'src/providers/proxy')
0 files changed, 0 insertions, 0 deletions