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

2017-07-03 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git

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

2017-07-03 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git

[SSSD] [sssd PR#300][comment] LDAP: Fix nesting level comparison

2017-07-03 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/300 Title: #300: LDAP: Fix nesting level comparison justin-stephenson commented: """ @fidencio thanks for the review, changes have been made. """ See the full comment at https://github.com/SSSD/sssd/pull/300#issuecomment-312715606

[SSSD] [sssd PR#300][synchronized] LDAP: Fix nesting level comparison

2017-07-03 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/300 Author: justin-stephenson Title: #300: LDAP: Fix nesting level comparison Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/300/head:pr300 git checkout pr300 From 961749f46e365f

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

2017-07-03 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: """ Implemented simple rhost verification (without DNS resolution yet), tested works as expected and checks passed. Would be nice to have a r

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

2017-07-03 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git

[SSSD] [sssd PR#300][comment] LDAP: Fix nesting level comparison

2017-07-03 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/300 Title: #300: LDAP: Fix nesting level comparison fidencio commented: """ @justin-stephenson: Thanks a lot for the really detailed explanation. So, patches look good. There's just one nitpick about the commit message on the patch touching the tests them

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

2017-07-03 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: """ Ah, that's fine then. My understanding was that all those fields are part of corresponding RFCs. If most of them aren't, then I guess no

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

2017-07-03 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option lslebodn commented: """ Probably, I still do not understand the problem. What do you mean by "to add new custom field"? Because most of attributes in `src/provide

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

2017-07-03 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: """ @lslebodn thanks for the pointer! Then the important question is it acceptable to add this new custom field that is outside of scope of R

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

2017-07-03 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/275 Title: #275: Implement access verification by rhost using ldap_access_order rhost option lslebodn commented: """ >Meanwhile I did not find how can I add extra LDAP field to be retrieved (with >default value) that would be outside of RFC scopes? Adding

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

2017-07-03 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: """ Updated the code to fix broken build. With this change it would use authorized_host field for rhost verification. Meanwhile I did not fi

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

2017-07-03 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: edited Changed field: body Original value: """ TL;DR - this is to implement functionality similar to both of `sshd_config:AllowUsers`

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

2017-07-03 Thread akamensky
URL: https://github.com/SSSD/sssd/pull/275 Author: akamensky Title: #275: Implement access verification by rhost using ldap_access_order rhost option Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/275/head:pr275 git