[SSSD] [sssd PR#189][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/189 Title: #189: SELINUX: Use getseuserbyname to get IPA seuser jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/189#issuecomment-285601383 ___ sssd-devel mailing l

[SSSD] [sssd PR#189][synchronized] SELINUX: Use getseuserbyname to get IPA seuser

2017-03-09 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/189 Author: justin-stephenson Title: #189: SELINUX: Use getseuserbyname to get IPA seuser Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/189/head:pr189 git checkout pr189 From 50

[SSSD] [sssd PR#189][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-03-09 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/189 Title: #189: SELINUX: Use getseuserbyname to get IPA seuser centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/189#issuecomment-285506518

[SSSD] [sssd PR#189][comment] SELINUX: Use getseuserbyname to get IPA seuser

2017-03-09 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/189 Title: #189: SELINUX: Use getseuserbyname to get IPA seuser centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/189#issuecomment-285506515

[SSSD] [sssd PR#189][opened] SELINUX: Use getseuserbyname to get IPA seuser

2017-03-09 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/189 Author: justin-stephenson Title: #189: SELINUX: Use getseuserbyname to get IPA seuser Action: opened PR body: """ Retrieve SELinux username utilizing libselinux API as a more reliable method than libsemanage calls and remove get_seuser function which

[SSSD] [sssd PR#174][comment] One certificate for multiple users

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/174 Title: #174: One certificate for multiple users jhrozek commented: """ On Thu, Mar 09, 2017 at 04:30:48AM -0800, sumit-bose wrote: > Thank you for testing. The code looking up overrides for IPA users was not > prepared to handle multiple results. I add

[SSSD] [sssd PR#154][+Changes requested] Confront caches first

2017-03-09 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/154 Title: #154: Confront caches first 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#188][comment] Add the first version of the KCM responder

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/188 Title: #188: Add the first version of the KCM responder jhrozek commented: """ CI: http://sssd-ci.duckdns.org/logs/job/64/32/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/188#issuecomment-285362951

[SSSD] [sssd PR#183][comment] More socket-activation fixes

2017-03-09 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/183 Title: #183: More socket-activation fixes fidencio commented: """ Patch set has been updated. Please, take a look on the commit messages for a good explanation (mainly for the "Avoid TOCTOU vulnerability with the log files" patch). """ See the full c

[SSSD] [sssd PR#183][-Changes requested] More socket-activation fixes

2017-03-09 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/183 Title: #183: More socket-activation fixes 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#183][synchronized] More socket-activation fixes

2017-03-09 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/183 Author: fidencio Title: #183: More socket-activation fixes Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/183/head:pr183 git checkout pr183 From f1683d696fe257bebd7f69393b971

[SSSD] [sssd PR#154][comment] Confront caches first

2017-03-09 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/154 Title: #154: Confront caches first pbrezina commented: """ Hi, thank you. This version looks much better. There where few logic issues in the code, I started to write the review but then I had a need to actually try to code my suggestions to see if it

[SSSD] [sssd PR#174][comment] One certificate for multiple users

2017-03-09 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/174 Title: #174: One certificate for multiple users sumit-bose commented: """ Thank you for testing. The code looking up overrides for IPA users was not prepared to handle multiple results. I added a patch to handle this at least for lookups by certificat

[SSSD] [sssd PR#174][-Changes requested] One certificate for multiple users

2017-03-09 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/174 Title: #174: One certificate for multiple users Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.or

[SSSD] [sssd PR#174][synchronized] One certificate for multiple users

2017-03-09 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/174 Author: sumit-bose Title: #174: One certificate for multiple users Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/174/head:pr174 git checkout pr174 From 321ec1ea6182f9f836888

[SSSD] [sssd PR#188][comment] Add the first version of the KCM responder

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/188 Title: #188: Add the first version of the KCM responder jhrozek commented: """ btw the first patches with the tevent wrapper and the iobuf module were also submitted separately. I'm still fighting with some CI issues in the other PR, but a review can

[SSSD] [sssd PR#152][synchronized] Add a tevent wrapper around libcurl's asynchronous interface

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/152 Author: jhrozek Title: #152: Add a tevent wrapper around libcurl's asynchronous interface Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/152/head:pr152 git checkout pr152 Fro

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

2017-03-09 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: opened PR body: """ The patches are rebased on top of #154 and provide a solution for https://pagure.io/SSSD/sssd/issue/3001 There

[SSSD] [sssd PR#186][opened] Subdomain config

2017-03-09 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/186 Author: mzidek-rh Title: #186: Subdomain config Action: opened PR body: """ This is the part of subdomain configuration feature needed to allow specify the search bases for the subdomain (one of the two use cases that were specified in the desin doc

[SSSD] [sssd PR#152][synchronized] Add a tevent wrapper around libcurl's asynchronous interface

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/152 Author: jhrozek Title: #152: Add a tevent wrapper around libcurl's asynchronous interface Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/152/head:pr152 git checkout pr152 Fro

[SSSD] [sssd PR#152][synchronized] Add a tevent wrapper around libcurl's asynchronous interface

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/152 Author: jhrozek Title: #152: Add a tevent wrapper around libcurl's asynchronous interface Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/152/head:pr152 git checkout pr152 Fro

[SSSD] [sssd PR#152][synchronized] Add a tevent wrapper around libcurl's asynchronous interface

2017-03-09 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/152 Author: jhrozek Title: #152: Add a tevent wrapper around libcurl's asynchronous interface Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/152/head:pr152 git checkout pr152 Fro

[SSSD] Re: [PATCH]: test ldap provider with TLS or SSL

2017-03-09 Thread Dan Lavu
Nik, Thanks for the review, the responses are inline. On Wed, Mar 8, 2017 at 1:59 AM, Nikolai Kondrashov < nikolai.kondras...@redhat.com> wrote: > On 03/07/2017 05:05 PM, Dan Lavu wrote: > >> Updated. >> > > Thanks! Please see my comments below. > > On 03/07/2017 05:05 PM, Dan Lavu wrote: > >> d