[sssd PR#582][opened] ipa: MPG realted fixes for lookups by GID

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/582
Author: sumit-bose
 Title: #582: ipa: MPG realted fixes for lookups by GID
Action: opened

PR body:
"""
There are a few issues when a trusted AD user with an expired cache entry is
indirectly looked up be a GID lookup for the primary user private group (mpg).

One if the issues was that sysdb_search_group_by_gid() is not aware of MPGs in
contrast to sysdb_search_group_by_name(). Since sysdb_search_group_by_gid() is
used at other places as well I added sysdb_getgrgid_attrs() to replace
sysdb_search_group_by_gid() in get_object_from_cache() instead of modifying to
avoid regressions in the other callers. Maybe it would be worth a ticket to
check if MPG support can be added safely to sysdb_search_group_by_gid().

Related to https://pagure.io/SSSD/sssd/issue/3748
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/582/head:pr582
git checkout pr582

[sssd PR#582][synchronized] ipa: MPG realted fixes for lookups by GID

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/582
Author: sumit-bose
 Title: #582: ipa: MPG realted fixes for lookups by GID
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/582/head:pr582
git checkout pr582

[sssd PR#582][comment] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

jhrozek commented:
"""
I tested several combinations of looking up the user, their group, other groups etc on both an IPA server and a directly enrolled client. I didn't see any regressions, Coverity was clean as well.
"""

See the full comment at https://github.com/SSSD/sssd/pull/582#issuecomment-395539070

[sssd PR#582][comment] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

jhrozek commented:
"""
(The accepted label is pending a CI run)
"""

See the full comment at https://github.com/SSSD/sssd/pull/582#issuecomment-395539121

[sssd PR#582][comment] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

jhrozek commented:
"""
CI: http://vm-031.ABC/logs/job/90/17/summary.html
"""

See the full comment at https://github.com/SSSD/sssd/pull/582#issuecomment-395679152

[sssd PR#582][+Accepted] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

Label: +Accepted

[sssd PR#582][comment] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

jhrozek commented:
"""
* master:
 * ad6ab352879264fdade8861aff53aa035a2e2240
 * e66517dcf63f1d4aaf866c22371dac7740ce0a48
 * 032221568fe4287686d0ebb11b5c1fe51cc4735f
 * 8aa56a9e8744a7611fa26a254c4f9228e919c8ed
"""

See the full comment at https://github.com/SSSD/sssd/pull/582#issuecomment-395764636

[sssd PR#582][closed] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Author: sumit-bose
 Title: #582: ipa: MPG realted fixes for lookups by GID
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/582/head:pr582
git checkout pr582

[sssd PR#582][+Pushed] ipa: MPG realted fixes for lookups by GID

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/582
Title: #582: ipa: MPG realted fixes for lookups by GID

Label: +Pushed