URL: https://github.com/SSSD/sssd/pull/5470 Author: sumit-bose Title: #5470: pam: refresh certificate maps at the end of initial domains lookup Action: opened
PR body: """ During startup SSSD's responders send a getDomains request to all backends to refresh some domain related needed by the responders.
The PAM responder specifically needs the certificate mapping and matching rules when Smartcard authentication is enable. Currently the rules are not refreshed at the end of the initial request but the code assumed that the related structures are initialized after the request finished.
To avoid a race condition this patch adds a callback to the end of the request to make sure the rules are properly refreshed even if they are already initialized before.
Resolves: https://github.com/SSSD/sssd/issues/5469 """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5470/head:pr5470 git checkout pr5470
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: +Waiting for review
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: +Bugzilla
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
thalman commented: """ The patch looks good, thank you ACK """
See the full comment at https://github.com/SSSD/sssd/pull/5470#issuecomment-765517903
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: -Waiting for review
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: +Ready to push
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5470
* `master` * cb936e92041d63f79a74c30bae8140c74a18dbc0 - pam: refresh certificate maps at the end of initial domains lookup * e07eeea7df55ede36ac0978ac904c1bb11188265 - responders: add callback to schedule_get_domains_task()
"""
See the full comment at https://github.com/SSSD/sssd/pull/5470#issuecomment-766718256
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: -Accepted
URL: https://github.com/SSSD/sssd/pull/5470 Title: #5470: pam: refresh certificate maps at the end of initial domains lookup
Label: -Ready to push
URL: https://github.com/SSSD/sssd/pull/5470 Author: sumit-bose Title: #5470: pam: refresh certificate maps at the end of initial domains lookup Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5470/head:pr5470 git checkout pr5470
sssd-devel@lists.fedorahosted.org