[SSSD] [sssd PR#378][comment] [RFC] Use GNULIB's compiler warning code

2017-09-27 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/378 Title: #378: [RFC] Use GNULIB's compiler warning code fidencio commented: """ Also, I'd like to open at least one bug to fix some of the warnings we've been ignoring so far. """ See the full comment at https://github.com/SSSD/sssd/pull/378#issuecomme

[SSSD] [sssd PR#374][-Changes requested] IPA: Add threshold for sudo command and command group searches

2017-09-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/374 Title: #374: IPA: Add threshold for sudo command and command group searches Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel

[SSSD] [sssd PR#378][comment] [RFC] Use GNULIB's compiler warning code

2017-09-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/378 Title: #378: [RFC] Use GNULIB's compiler warning code jhrozek commented: """ @fidencio I'm sorry this stalled, but I guess the PR needs a rebase atop master, right? At least the fixes for the warnings are present in master now.. """ See the full comme

[SSSD] [sssd PR#386][comment] intg: prevent "TypeError: must be type, not classobj"

2017-09-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/386 Title: #386: intg: prevent "TypeError: must be type, not classobj" jhrozek commented: """ @amitkumar50 can you please change the title of this PR so that we can push the patch? Thank you! """ See the full comment at https://github.com/SSSD/sssd/pull/

[SSSD] [sssd PR#390][-Changes requested] NSS: Add option to disable memcache

2017-09-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.o

[SSSD] [sssd PR#390][comment] NSS: Add option to disable memcache

2017-09-27 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/390 Title: #390: NSS: Add option to disable memcache jhrozek commented: """ There is a *really* big difference between an option that leaks user's credentials to anyone sniffing the network and an option that lets you decrease NSS lookups performance. Esp

[SSSD] [sssd PR#275][comment] Implement access verification by rhost using ldap_access_order rhost option

2017-09-27 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option akamensky commented: """ I am thinking if can first merge what it is now and then add hostname resolution in the next release. I already have some work for resolu

[SSSD] [sssd PR#275][comment] Implement access verification by rhost using ldap_access_order rhost option

2017-09-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option pbrezina commented: """ Code-wise ack. I still need to test it, but I will do that probably in the second half of October since I'll be on PTO. Do you also plan o

[SSSD] [sssd PR#275][comment] Implement access verification by rhost using ldap_access_order rhost option

2017-09-27 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option pbrezina commented: """ I have no problem with putting it into sdap_access tests. """ See the full comment at https://github.com/SSSD/sssd/pull/275#issuecomment-