[sssd PR#252][opened] PAM: check matching certificates from all domains

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/252
Author: sumit-bose
 Title: #252: PAM: check matching certificates from all domains
Action: opened

PR body:
"""
Although the cache_req lookup found matching in multiple domains only
the results from the first domain were used. With this patch the results
from all domains are checked.

Resolves https://pagure.io/SSSD/sssd/issue/3385
"""

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

[sssd PR#252][comment] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

jhrozek commented:
"""
The code looks good to me, I'm just waiting for coverity results

@sumit-bose did the original reporter of the patch confirm that it works? I only checked the code
"""

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

[sssd PR#252][comment] PAM: check matching certificates from all domains

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

sumit-bose commented:
"""
Yes, it was confirmed that the patch solves the issue.
"""

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

[sssd PR#252][comment] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

jhrozek commented:
"""
thank you, ACK, the Coverity scan didn't find any issues either.
"""

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

[sssd PR#252][+Accepted] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

Label: +Accepted

[sssd PR#252][comment] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

jhrozek commented:
"""
* master: 92d8b072f8c521e1b4effe109b5caedabd36ed6f
"""

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

[sssd PR#252][closed] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Author: sumit-bose
 Title: #252: PAM: check matching certificates from all domains
Action: closed

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

[sssd PR#252][+Pushed] PAM: check matching certificates from all domains

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/252
Title: #252: PAM: check matching certificates from all domains

Label: +Pushed