[SSSD] [sssd PR#713][opened] krb5_child: fix permissions during SC auth

2018-12-11 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/713 Author: sumit-bose Title: #713: krb5_child: fix permissions during SC auth Action: opened PR body: """ For PKINIT we might need access to the pcscd socket which by default is only allowed for authenticated users. Since PKINIT is part of the authentica

[SSSD] [sssd PR#703][closed] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/703/head:pr703 git checkout pr703 _

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ * sssd-1-16: 28792523a01a7d21bcc8931794164f253e691a68 """ See the full comment at https://github.com/SSSD/sssd/pull/703#issuecomment-446378330 ___

[SSSD] [sssd PR#703][+Pushed] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ * master: 90f32399b4100ce39cf665649fde82d215e5eb49 """ See the full comment at https://github.com/SSSD/sssd/pull/703#issuecomment-446378006 __

[SSSD] [sssd PR#711][comment] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root jhrozek commented: """ * sssd-1-16: * 74568bdde833f752187cb1a38b39715556c91279 * d33ec64423087261fcc14acb5922793fadb83342 """ See the full comment at https://github.com/SSSD/ss

[SSSD] [sssd PR#711][+Pushed] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.f

[SSSD] [sssd PR#711][closed] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Author: sumit-bose Title: #711: ipa: use only the global catalog service of the forest root Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/711/head:pr711 git checkout pr711 ___

[SSSD] [sssd PR#711][comment] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root jhrozek commented: """ * master: * 62d671b874a66101c0f4bff39fc6d7f49cb8fca6

[SSSD] [sssd PR#677][comment] pcre: port to pcre2

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: pcre: port to pcre2 jhrozek commented: """ I'm sorry to keep beating the error code from *sss_regexp_new...but why not just return int from that function, which would be EOK on success, in which case a **self pointer would also be returne

[SSSD] [sssd PR#705][-Changes requested] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedo

[SSSD] [sssd PR#705][synchronized] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Author: jhrozek Title: #705: KCM: Add configurable quotas Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/705/head:pr705 git checkout pr705 From aeaa27423ee1b5a70f556a937bc450

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ OK, I'll squash this diff: ``` diff --git a/src/responder/secrets/secsrv.c b/src/responder/secrets/secsrv.c

[SSSD] [sssd PR#711][+Accepted] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists

[SSSD] [sssd PR#705][+Changes requested] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedo

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ I'm adding changes requested, but since the issue is 'only' in tests, I would still appreciate a review """ See the full comment at https://github.com/SSSD/sssd/pull/705#issuecomment-44

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ I still need to amend the tests: ``` === short test summary info FAIL test_kcm.py::test_kcm_secrets_quota FAIL test_secrets.py::test_gl

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread simo5
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories simo5 commented: """ Thank @jhrozek this clears it! """ See the full comment at https://github.com/SSSD/sssd/pull/703#issuecomment-446204376 ___

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ The patch does change the behaviour, but it's also just a fallback, so whatever you had defined in AD LDAP is still used. Let me give an example: - before the patch:

[SSSD] [sssd PR#710][synchronized] data_provider_fo: fix error in hostname retrieval

2018-12-11 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/710 Author: alexey-tikhonov Title: #710: data_provider_fo: fix error in hostname retrieval Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/710/head:pr710 git checkout pr710 From 0

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread simo5
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories simo5 commented: """ Wait, does this mean it changes current behavior for AD domains ? """ See the full comment at https://github.com/SSSD/sssd/pull/703#issuecomment-446198589 _

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread simo5
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories simo5 commented: """ Or would it previously return "/" unconditionally ? """ See the full comment at https://github.com/SSSD/sssd/pull/703#issuecomment-446198697 ___

[SSSD] [sssd PR#711][synchronized] ipa: use only the global catalog service of the forest root

2018-12-11 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/711 Author: sumit-bose Title: #711: ipa: use only the global catalog service of the forest root Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/711/head:pr711 git checkout pr711 F

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/705#issuecomment-446184702 ___ sssd-devel mailing list -- sss

[SSSD] [sssd PR#712][opened] SSSCTL: user-checks does not show custom attributes

2018-12-11 Thread thalman
URL: https://github.com/SSSD/sssd/pull/712 Author: thalman Title: #712: SSSCTL: user-checks does not show custom attributes Action: opened PR body: """ "sssctl user-checks" also query the IFP interface of sssd. The problem is, that this command only displays POSIX user attrs, no matter if we

[SSSD] [sssd PR#703][+Accepted] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahost

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ Seems to work fine, by default I get /home/domain/username for all admins, when I set fallback_homedir=%o then the unixHomeDirectory attribute is used instead. """ Se