URL: https://github.com/SSSD/sssd/pull/207 Author: sumit-bose Title: #207: nss-idmap: add sss_nss_getlistbycert() Action: opened
PR body: """ This patch adds a getlistbycert() call to libsss_nss_idmap to make it on par with InfoPipe.
Related to https://pagure.io/SSSD/sssd/issue/3050
The easiest way to test is via the python bindings with
python -c "import pysss_nss_idmap; print pysss_nss_idmap.getlistbycert('MIIFvz....')"
where the argument is the base64 encoded certificate. """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/207/head:pr207 git checkout pr207
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
jhrozek commented: """ CI passed: http://sssd-ci.duckdns.org/logs/job/65/30/summary.html
Coverity is also clean """
See the full comment at https://github.com/SSSD/sssd/pull/207#issuecomment-288759704
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
jhrozek commented: """ The code works fine: ```
print pysss_nss_idmap.getlistbycert("MIIC8zCCAdugAwIBAgIJAPktuv1fxruLMA0GCSqGSIb3DQEBCwUAMBAxDjAMBgNVBAMMBXR1c2VyMB4XDTE3MDMwODEwMzU1MFoXDTE4MDMwODEwMzU1MFowEDEOMAwGA1UEAwwFdHVzZXIwggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQCuqwIDSxK06tgeW5NKHAnpjElosrV4QkxA9XcG9JvLfVhZLPR2NZlzWRYUUGf/B1up7YbpPOLiFp6UsV/IziHQpvB+ENg6JyPJvgT+ylEKQpaKUqg3qPp5u7yw+h5RfhnqtzzsxdvYUfhBHb/D7/Rflm0aD/6GpOl/oEDC71rA7CV4nRisrJGuS9QUiuADvcf8ys7GddkphuNRp3r8J3TLv25YibrFHfvr455/EWztulrRP8pLBVwe7BYiy74Q01nbyw2bWDHa1kvJ97BEQMyYHi9jePKheuatIg00rIuw6z33HAQeXu9Eq259mvOTcsgHQY1x7T9QAZapvZWiJRNtAgMBAAGjUDBOMB0GA1UdDgQWBBQ/zxIB0eBccxbVa4ZAPFtFwEClbTAfBgNVHSMEGDAWgBQ/zxIB0eBccxbVa4ZAPFtFwEClbTAMBgNVHRMEBTADAQH/MA0GCSqGSIb3DQEBCwUAA4IBAQAtJqzpVgxv8DnrwGUfImuMRv58hZRmmswniNu2AhSm4jnTKyUrBubDQRTyPOSs8lDSzgAARfxueQZ357t3+gLI+15pb5oAOos5uzOiRyBL81DUq8gKLpWIk4OnhsGR8cfzp8YQI7i4/8k00Gc+JmH6za2m67ddgWnfIOv4js+P+1OUgTP1HfD5eL6lRQMqgaScJB+PtkUznm7qzzRvw9FQ1fRe0FjGVoxRm2lhioRaasieQiCBKHKMWlVvLnMbTvov7foPPB5Rmj1p59M+xltrTJQSE6XORJCvOlvtWNrNmGjRZKAK5hpxifzIa6SWquT+PS/BIjzkNN8D7Tcnq7zk")
{'MIIC8zCCAdugAwIBAgIJAPktuv1fxruLMA0GCSqGSIb3DQEBCwUAMBAxDjAMBgNVBAMMBXR1c2VyMB4XDTE3MDMwODEwMzU1MFoXDTE4MDMwODEwMzU1MFowEDEOMAwGA1UEAwwFdHVzZXIwggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQCuqwIDSxK06tgeW5NKHAnpjElosrV4QkxA9XcG9JvLfVhZLPR2NZlzWRYUUGf/B1up7YbpPOLiFp6UsV/IziHQpvB+ENg6JyPJvgT+ylEKQpaKUqg3qPp5u7yw+h5RfhnqtzzsxdvYUfhBHb/D7/Rflm0aD/6GpOl/oEDC71rA7CV4nRisrJGuS9QUiuADvcf8ys7GddkphuNRp3r8J3TLv25YibrFHfvr455/EWztulrRP8pLBVwe7BYiy74Q01nbyw2bWDHa1kvJ97BEQMyYHi9jePKheuatIg00rIuw6z33HAQeXu9Eq259mvOTcsgHQY1x7T9QAZapvZWiJRNtAgMBAAGjUDBOMB0GA1UdDgQWBBQ/zxIB0eBccxbVa4ZAPFtFwEClbTAfBgNVHSMEGDAWgBQ/zxIB0eBccxbVa4ZAPFtFwEClbTAMBgNVHRMEBTADAQH/MA0GCSqGSIb3DQEBCwUAA4IBAQAtJqzpVgxv8DnrwGUfImuMRv58hZRmmswniNu2AhSm4jnTKyUrBubDQRTyPOSs8lDSzgAARfxueQZ357t3+gLI+15pb5oAOos5uzOiRyBL81DUq8gKLpWIk4OnhsGR8cfzp8YQI7i4/8k00Gc+JmH6za2m67ddgWnfIOv4js+P+1OUgTP1HfD5eL6lRQMqgaScJB+PtkUznm7qzzRvw9FQ1fRe0FjGVoxRm2lhioRaasieQiCBKHKMWlVvLnMbTvov7foPPB5Rmj1p59M+xltrTJQSE6XORJCvOlvtWNrNmGjRZKAK5hpxifzIa6SWquT+PS/BIjzkNN8D7Tcnq7zk': [{'type': 1, 'name': u'anotheruser@ipa.test'}, {'type': 1, 'name': u'tuser@ipa.test'}]} ```
And also looks OK to me, ACK """
See the full comment at https://github.com/SSSD/sssd/pull/207#issuecomment-288771713
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
jhrozek commented: """ * master: * a0b1bfa76073d3ce3208e67e6d72bb92088edac5 * 440797cba931aa491bf418035f55935943e22b4b
"""
See the full comment at https://github.com/SSSD/sssd/pull/207#issuecomment-288776089
URL: https://github.com/SSSD/sssd/pull/207 Author: sumit-bose Title: #207: nss-idmap: add sss_nss_getlistbycert() Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/207/head:pr207 git checkout pr207
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
Label: +Pushed
URL: https://github.com/SSSD/sssd/pull/207 Title: #207: nss-idmap: add sss_nss_getlistbycert()
Label: -Accepted
sssd-devel@lists.fedorahosted.org