[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ @pbrezina: patchset updated. About setting a bigger value for the default subdomain timeout, it can be discussed out of this PR

[SSSD] [sssd PR#187][synchronized] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Author: fidencio Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/187/head:pr187

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder jhrozek commented: """ On Tue, Mar 21, 2017 at 10:31:10AM -0700, mzidek-rh wrote: > Nitpick: In the contrib/kcm_default_ccache it would be good to indicate (in > the comments) where the snippet belongs. Done. > It is

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder mzidek-rh commented: """ Nitpick: In the contrib/kcm_default_ccache it would be good to indicate (in the comments) where the snippet belongs. It is written in the commit message, but if someone just opens the file he/she

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder jhrozek commented: """ On Tue, Mar 21, 2017 at 09:26:23AM -0700, mzidek-rh wrote: > Actually we do check for libcurl in configure. The reproducer is: > > > $ sudo dnf remove libcurl-devel > $ configure | grep curl >

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder mzidek-rh commented: """ Actually we do check for libcurl in configure. The reproducer is: $ sudo dnf remove libcurl-devel $ configure | grep curl configure: WARNING: The libcurl development library was not found. Some

[SSSD] [sssd PR#192][comment] Add certificate mapping library

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/192 Title: #192: Add certificate mapping library jhrozek commented: """ I won't pretend I read the certmap library code carefully, I more or less checked the API, ran the tests and scrolled through the library code. My only concern there is the memory

[SSSD] [sssd PR#192][+Changes requested] Add certificate mapping library

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/192 Title: #192: Add certificate mapping library 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#192][comment] Add certificate mapping library

2017-03-21 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/192 Title: #192: Add certificate mapping library lslebodn commented: """ On (21/03/17 08:59), Jakub Hrozek wrote: >jhrozek commented on this pull request. >> +src/util/crypto/nss/nss_base64.c \ >+src/util/cert/nss/cert.c \ >+

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder mzidek-rh commented: """ On VM with were libcurl-devel is not installed, the configure succeeds, but compilation fails with: /home/user/gitrepo/sssd/src/util/tev_curl.c:32:23: fatal error: curl/curl.h: No such file or

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder mzidek-rh commented: """ On VM with were libcurl-devel is not installed, the configure succeeds, but compilation fails with: /home/user/gitrepo/sssd/src/util/tev_curl.c:32:23: fatal error: curl/curl.h: No such file or

[SSSD] [sssd PR#197][comment] KCM responder

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder jhrozek commented: """ I pushed new patches that implement what Simo asked for: * the kcm_ops module now queues all requests from the same UID * the secrets module does no special casing of the KCM paths * the KCM data is

[SSSD] [sssd PR#197][-Changes requested] KCM responder

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder 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#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname pbrezina commented: """ There may be someway to check if the configuration has changed. Or we can increase the default subdomain timeout from 60 seconds to

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ On Tue, Mar 21, 2017 at 1:30 PM, Pavel Březina wrote: > Function create_domains_lookup_order_list needs to

[SSSD] [sssd PR#187][+Changes requested] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname pbrezina commented: """ Function `create_domains_lookup_order_list` needs to be called in `sss_dp_get_domains_process` prior calling `tevent_req_done`

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname fidencio commented: """ New patchset pushed, the fixup changes look like: ``` diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml index

[SSSD] [sssd PR#187][synchronized] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/187 Author: fidencio Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/187/head:pr187

[SSSD] [sssd PR#205][+Accepted] MAN: Mention sssd-secrets in "SEE ALSO" section

2017-03-21 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/205 Title: #205: MAN: Mention sssd-secrets in "SEE ALSO" section Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#205][opened] MAN: Mention sssd-secrets in "SEE ALSO" section

2017-03-21 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/205 Author: lslebodn Title: #205: MAN: Mention sssd-secrets in "SEE ALSO" section Action: opened PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/3344 """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch

[SSSD] [sssd PR#187][comment] Add support to lookup for users/groups in subdomains just by the user shortname

2017-03-21 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/187 Title: #187: Add support to lookup for users/groups in subdomains just by the user shortname sumit-bose commented: """ Hi Fabiano, thank you for the patch, it looks good, I only added two minor comments. I didn't had a chance to test it yet. """ See

[SSSD] [sssd PR#204][opened] krb5: return to responder that pkinit is not available

2017-03-21 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/204 Author: sumit-bose Title: #204: krb5: return to responder that pkinit is not available Action: opened PR body: """ If pkinit is not available for a user but other authentication methods are SSSD should still fall back to local certificate based

[SSSD] [sssd PR#139][comment] Initial revision of sssd pytest framework

2017-03-21 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework lslebodn commented: """ On (21/03/17 01:02), Niranjan Mallapadi wrote: >Any update on this ? > It's on my TODO list. But unfortunatelly downstream has higher priority ATM. LS """ See the full

[SSSD] [sssd PR#197][+Changes requested] KCM responder

2017-03-21 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/197 Title: #197: KCM responder 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#139][comment] Initial revision of sssd pytest framework

2017-03-21 Thread mrniranjan
URL: https://github.com/SSSD/sssd/pull/139 Title: #139: Initial revision of sssd pytest framework mrniranjan commented: """ Any update on this ? """ See the full comment at https://github.com/SSSD/sssd/pull/139#issuecomment-288003254 ___ sssd-devel