#3392 org.freedesktop.sssd.infopipe.GetUserGroups does not resolve groups into names with AD
Closed: Fixed 6 years ago Opened 6 years ago by jhrozek.

The AD provider only converts SIDs to GIDs during initgroups to improve performance. But this is not sufficient for the org.freedesktop.sssd.infopipe.GetUserGroups method, which needs to return names.

We need to resolve the GIDs to names ourselves in that method.


@jhrozek, AFAIR it could be done cache_req side as we may already have the groupname info when the groups are fetched.

I mean, we would still have to do have some logic add to cache_req, but I do believe it could be done there directly.

I'm not sure, because the cache_req doesn't know if the caller is really calling initgroups because he's interested in the usual initgroups result (=a list of GIDs) or whether the caller is interested in names as well.

Anyway, my solution is at https://github.com/SSSD/sssd/compare/master...jhrozek:ifp_groupnames

Metadata Update from @jhrozek:
- Issue assigned to jhrozek

6 years ago

Metadata Update from @jhrozek:
- Custom field patch adjusted to on

6 years ago

Metadata Update from @jhrozek:
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1449729

6 years ago

Metadata Update from @jhrozek:
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1449729

6 years ago

Metadata Update from @jhrozek:
- Issue set to the milestone: SSSD 1.15.3

6 years ago

Metadata Update from @jhrozek:
- Issue priority set to: critical

6 years ago

sssd-1-14:

Previous patches were backported by mistake. There are missing cache_req features in 1.14. Patches were reverted in 1.14 branch

sssd-1-14:

Metadata Update from @lslebodn:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4419

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata