[SSSD] getaddrinfo() may fail with AI_ADDRCONFIG

2018-06-05 Thread Amit
Hello, AI_ADDRCONFIG was added in order to optimise DNS query traffic, so that only useful addresses are queried for. In other words, an IPv4-only node should not query its upstream resolver for IN resource records, while an IPv6-only node should not query for IN A resource records. The

[SSSD] [sssd PR#590][comment] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved fidencio commented: """ Ack! I'll fire our internal CI and add the "Accepted" label after I get the results. There's just one question, though: IIRC we do the

[SSSD] [sssd PR#591][comment] Remove spaces after comment delimiters in the example config file

2018-06-05 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: Remove spaces after comment delimiters in the example config file centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/591#issuecomment-394635844

[SSSD] Re: https://pagure.io/SSSD/sssd/issue/1555 [validate the shell coming from the directory or the cache] Close or not?

2018-06-05 Thread Jakub Hrozek
> On 5 Jun 2018, at 07:53, Amit wrote: > > Hello, > > Issue was opened 5 years back with following description: It is possible to > put junk into the shell attribute of an user entry. We should reuse the > existing code that is in use when allowed_shells/vetoed_shells are present, > check

[SSSD] [sssd PR#591][comment] Remove spaces after comment delimiters in the example config file

2018-06-05 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: Remove spaces after comment delimiters in the example config file centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/591#issuecomment-394635843

[SSSD] [sssd PR#590][comment] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved jhrozek commented: """ > On 5 Jun 2018, at 09:16, fidencio wrote: > > Ack! > I'll fire our internal CI and add the "Accepted" label after I get the >

[SSSD] [sssd PR#591][comment] Remove spaces after comment delimiters in the example config file

2018-06-05 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: Remove spaces after comment delimiters in the example config file sumit-bose commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/591#issuecomment-394649795

[SSSD] [sssd PR#589][comment] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default fidencio commented: """ Patch looks good and I've fired an internal CI build. I'll add the "Accepted" label as soon as I get CI results. """ See the full comment at

[SSSD] [sssd PR#585][comment] tcurl: do not log the payload

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/585 Title: #585: tcurl: do not log the payload fidencio commented: """ I have updated the patch with a stopgap fix. """ See the full comment at https://github.com/SSSD/sssd/pull/585#issuecomment-394688792 ___

[SSSD] [sssd PR#585][-Changes requested] tcurl: do not log the payload

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/585 Title: #585: tcurl: do not log the payload Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#585][synchronized] tcurl: do not log the payload

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/585 Author: fidencio Title: #585: tcurl: do not log the payload Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/585/head:pr585 git checkout pr585 From

[SSSD] [sssd PR#590][comment] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved fidencio commented: """ CI: http://vm-031.${abc}/logs/job/89/93/summary.html """ See the full comment at

[SSSD] [sssd PR#590][+Accepted] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#589][comment] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default fidencio commented: """ Ack! CI: http://vm-031.${abc}/logs/job/89/94/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/589#issuecomment-394750016

[SSSD] [sssd PR#589][+Accepted] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#579][comment] krb5 locator: add support for multiple addresses and sub-domains

2018-06-05 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/579 Title: #579: krb5 locator: add support for multiple addresses and sub-domains sumit-bose commented: """ Hi Jakub, thank you for the review I added a new patch which make the removal of the krb5info files more flexible and takes care of the removal of

[SSSD] [sssd PR#579][synchronized] krb5 locator: add support for multiple addresses and sub-domains

2018-06-05 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/579 Author: sumit-bose Title: #579: krb5 locator: add support for multiple addresses and sub-domains Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/579/head:pr579 git checkout

[SSSD] [sssd PR#579][-Changes requested] krb5 locator: add support for multiple addresses and sub-domains

2018-06-05 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/579 Title: #579: krb5 locator: add support for multiple addresses and sub-domains Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#589][comment] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default fidencio commented: """ master: 9adc750 """ See the full comment at https://github.com/SSSD/sssd/pull/589#issuecomment-394829312 ___

[SSSD] [sssd PR#589][closed] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Author: jhrozek Title: #589: RESPONDERS: Enable the local negative timeout by default Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/589/head:pr589 git checkout pr589

[SSSD] [sssd PR#589][+Pushed] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#589][-Accepted] RESPONDERS: Enable the local negative timeout by default

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/589 Title: #589: RESPONDERS: Enable the local negative timeout by default Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#570][comment] p11_child: add OpenSSL support

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/570 Title: #570: p11_child: add OpenSSL support fidencio commented: """ master: 2f897af 176e4d2 842daeb 4f63a1a 7190e0e 165f58a b5136cd 6d6e4a5 4eed225 075f2f3 ee76c68 8adf6ea 8127b58 """ See the full comment at

[SSSD] [sssd PR#570][closed] p11_child: add OpenSSL support

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/570 Author: sumit-bose Title: #570: p11_child: add OpenSSL support Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/570/head:pr570 git checkout pr570

[SSSD] [sssd PR#570][+Pushed] p11_child: add OpenSSL support

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/570 Title: #570: p11_child: add OpenSSL support Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code

[SSSD] [sssd PR#570][-Accepted] p11_child: add OpenSSL support

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/570 Title: #570: p11_child: add OpenSSL support Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#585][comment] tcurl: do not log the payload

2018-06-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/585 Title: #585: tcurl: do not log the payload jhrozek commented: """ Ack. If someone can set an environment variable in SSSD there are more harmful things they can do than logging a TGT value into a file only readable by root. """ See the full comment

[SSSD] [sssd PR#585][+Accepted] tcurl: do not log the payload

2018-06-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/585 Title: #585: tcurl: do not log the payload Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code

[SSSD] [sssd PR#590][+Pushed] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#590][closed] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Author: jhrozek Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/590/head:pr590 git

[SSSD] [sssd PR#586][comment] SSH: Do not exit abruptly if SSHD closes its end of the pipe before reading all the SSH keys

2018-06-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/586 Title: #586: SSH: Do not exit abruptly if SSHD closes its end of the pipe before reading all the SSH keys jhrozek commented: """ this PR really does need an integration test and I don't want to rush it in before this week's release so I'm tagging it

[SSSD] [sssd PR#590][-Accepted] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#586][+Changes requested] SSH: Do not exit abruptly if SSHD closes its end of the pipe before reading all the SSH keys

2018-06-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/586 Title: #586: SSH: Do not exit abruptly if SSHD closes its end of the pipe before reading all the SSH keys Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To

[SSSD] [sssd PR#590][comment] LDAP: Suppress a loud debug message in case a built-in SID can't be resolved

2018-06-05 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/590 Title: #590: LDAP: Suppress a loud debug message in case a built-in SID can't be resolved fidencio commented: """ master: df8e105 """ See the full comment at https://github.com/SSSD/sssd/pull/590#issuecomment-394828626

[SSSD] [sssd PR#592][opened] too complex if-else structure in sss_dp_get_account_msg

2018-06-05 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/592 Author: amitkumar50 Title: #592: too complex if-else structure in sss_dp_get_account_msg Action: opened PR body: """ The structure of if-else statements in sss_dp_get_account_msg become too complex. Replacing if-else with switch initially, Will

[SSSD] [sssd PR#592][comment] too complex if-else structure in sss_dp_get_account_msg

2018-06-05 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/592 Title: #592: too complex if-else structure in sss_dp_get_account_msg centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/592#issuecomment-394949267

[SSSD] [sssd PR#592][comment] too complex if-else structure in sss_dp_get_account_msg

2018-06-05 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/592 Title: #592: too complex if-else structure in sss_dp_get_account_msg centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/592#issuecomment-394949266