[sssd PR#281][opened] ldap: handle certmap errors gracefully

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/281
Author: sumit-bose
 Title: #281: ldap: handle certmap errors gracefully
Action: opened

PR body:
"""
Currently the LDAP user lookup request errors out if e.g. there is no
matching rule for a certificate. This might cause the related domain to
go offline.

With this patch the request returns that no user was found for the given
certificate but overall result is that the request finishes
successfully.

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

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

[sssd PR#281][comment] ldap: handle certmap errors gracefully

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

jhrozek commented:
"""
Do we care about the case where there were rules but no longer are? IOW, should we also remove whatever there might have been previously in the cache?
"""

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

[sssd PR#281][comment] ldap: handle certmap errors gracefully

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

sumit-bose commented:
"""
Good point, I'll enhance the patch
"""

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

[sssd PR#281][+Changes requested] ldap: handle certmap errors gracefully

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

Label: +Changes requested

[sssd PR#281][synchronized] ldap: handle certmap errors gracefully

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/281
Author: sumit-bose
 Title: #281: ldap: handle certmap errors gracefully
Action: synchronized

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

[sssd PR#281][-Changes requested] ldap: handle certmap errors gracefully

[sumit-bose]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

Label: -Changes requested

[sssd PR#281][comment] ldap: handle certmap errors gracefully

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

jhrozek commented:
"""
CI: http://sssd-ci.duckdns.org/logs/job/70/56/summary.html
"""

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

[sssd PR#281][+Accepted] ldap: handle certmap errors gracefully

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

Label: +Accepted

[sssd PR#281][closed] ldap: handle certmap errors gracefully

[lslebodn]

URL: https://github.com/SSSD/sssd/pull/281
Author: sumit-bose
 Title: #281: ldap: handle certmap errors gracefully
Action: closed

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

[sssd PR#281][comment] ldap: handle certmap errors gracefully

[lslebodn]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

lslebodn commented:
"""
master:
* 29ee3e0945f8935a2eb01913ba00b540e0a94f01
"""

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

[sssd PR#281][+Pushed] ldap: handle certmap errors gracefully

[lslebodn]

URL: https://github.com/SSSD/sssd/pull/281
Title: #281: ldap: handle certmap errors gracefully

Label: +Pushed