[SSSD] [sssd PR#618][comment] Add an option to only print the host public keys for sss_ssh_knownhostsproxy

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/618 Title: #618: Add an option to only print the host public keys for sss_ssh_knownhostsproxy jhrozek commented: """ I'm sorry, but I'm getting a segfault with the simple case of requesting a key for a host: valgrind: ``` libtool --mode=execute valgrind

[SSSD] [sssd PR#618][+Changes requested] Add an option to only print the host public keys for sss_ssh_knownhostsproxy

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/618 Title: #618: Add an option to only print the host public keys for sss_ssh_knownhostsproxy Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an

[SSSD] [sssd PR#591][closed] sss_obuscate: Fix traceback while reading space in sssd.conf

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/591 Author: amitkumar50 Title: #591: sss_obuscate: Fix traceback while reading space in sssd.conf Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/591/head:pr591 git checkout pr591

[SSSD] [sssd PR#591][comment] sss_obuscate: Fix traceback while reading space in sssd.conf

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: sss_obuscate: Fix traceback while reading space in sssd.conf jhrozek commented: """ Since there was no reply for over a month, I'm closing this PR. Feel free to reopen later. """ See the full comment at

[SSSD] [sssd PR#591][+Rejected] sss_obuscate: Fix traceback while reading space in sssd.conf

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: sss_obuscate: Fix traceback while reading space in sssd.conf Label: +Rejected ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#622][comment] MC: Remove check if record is in the mapped address space

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/622 Title: #622: MC: Remove check if record is in the mapped address space jhrozek commented: """ @mzidek-rh can you review this patch, please? """ See the full comment at https://github.com/SSSD/sssd/pull/622#issuecomment-406637015

[SSSD] [sssd PR#622][opened] MC: Remove check if record is in the mapped address space

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/622 Author: jhrozek Title: #622: MC: Remove check if record is in the mapped address space Action: opened PR body: """ There is a check in the memory cache code that checks if a record pointer points to the mmapped region . But since some time ago, we

[SSSD] [sssd PR#621][comment] sdap: respect passwordGracelimit

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/621 Title: #621: sdap: respect passwordGracelimit jhrozek commented: """ before adding the accepted label I will run some downstream tests """ See the full comment at https://github.com/SSSD/sssd/pull/621#issuecomment-406625918

[SSSD] [sssd PR#621][synchronized] sdap: respect passwordGracelimit

2018-07-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/621 Author: fidencio Title: #621: sdap: respect passwordGracelimit Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/621/head:pr621 git checkout pr621 From

[SSSD] [sssd PR#568][+Changes requested] Strip whitespaces in netgroup triple.

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/568 Title: #568: Strip whitespaces in netgroup triple. Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#568][comment] Strip whitespaces in netgroup triple.

2018-07-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/568 Title: #568: Strip whitespaces in netgroup triple. jhrozek commented: """ btw I'm setting changes requested to this PR because the unused variables should be squashed. I can do that myself as well if you don't have the time (I hope you didn't get too

[SSSD] [sssd PR#621][synchronized] sdap: respect passwordGracelimit

2018-07-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/621 Author: fidencio Title: #621: sdap: respect passwordGracelimit Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/621/head:pr621 git checkout pr621 From

[SSSD] [sssd PR#620][comment] Add pam_cert_pam_services option

2018-07-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/620 Title: #620: Add pam_cert_pam_services option fidencio commented: """ @abbra, patch looks good to me. I'd also provide a scratch-build to Orion in order to be sure it works for him. In any case, if it's not urgent I'd also like to have either

[SSSD] [sssd PR#620][comment] Add pam_cert_pam_services option

2018-07-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/620 Title: #620: Add pam_cert_pam_services option fidencio commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/620#issuecomment-406566714 ___ sssd-devel mailing list --

[SSSD] [sssd PR#621][opened] sdap: respect passwordGracelimit

2018-07-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/621 Author: fidencio Title: #621: sdap: respect passwordGracelimit Action: opened PR body: """ Since recent changes in 389-ds two response controls are end when passwordGracelimit is set and about to expire: - [1.3.6.1.4.1.42.2.27.8.5.1] for the

[SSSD] [sssd PR#620][opened] Add pam_cert_pam_services option

2018-07-20 Thread abbra
URL: https://github.com/SSSD/sssd/pull/620 Author: abbra Title: #620: Add pam_cert_pam_services option Action: opened PR body: """ Allow customizing which PAM services are allowed to perform smartcard authentication. Fixes: https://pagure.io/SSSD/sssd/issue/3775 """ To pull the PR as Git

[SSSD] [sssd PR#620][comment] Add pam_cert_pam_services option

2018-07-20 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/620 Title: #620: Add pam_cert_pam_services option centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/620#issuecomment-406550777 ___

[SSSD] [sssd PR#620][comment] Add pam_cert_pam_services option

2018-07-20 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/620 Title: #620: Add pam_cert_pam_services option centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/620#issuecomment-406550776 ___