[SSSD] [sssd PR#506][closed] Fixed subdomain typo

2018-01-29 Thread lslebodn
   URL: https://github.com/SSSD/sssd/pull/506
Author: tscherf
 Title: #506: Fixed subdomain typo
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/506/head:pr506
git checkout pr506
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#506][+Pushed] Fixed subdomain typo

2018-01-29 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/506
Title: #506: Fixed subdomain typo

Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#506][comment] Fixed subdomain typo

2018-01-29 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/506
Title: #506: Fixed subdomain typo

lslebodn commented:
"""
master:
* d4a6579a99b8cdeab31f69dfa3b88c145dd9e845
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/506#issuecomment-361398053
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

lslebodn commented:
"""
The branch sssd-1-13 is quite diverged from master branch. Therefore it would 
be good to check all functions which use `ldb_search`. I did that for master 
branch as part of review of original fix.
Was it done also for 1.13 branch which is LTM ? We can double/triple check 
that. (I am a volunteer to do that but later this week)

(temporary removing ACK label)

BTW it would be good to mention in commit related ticket 
https://pagure.io/SSSD/sssd/issue/3549
but that can be done before pushing.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/496#issuecomment-361395540
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][-Accepted] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread lslebodn
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

lslebodn commented:
"""
> CI has passed apart from some issue running mock-build on rawhide: 
> http://vm-031.${abc}/logs/job/83/25/summary.html
>
> May the failure there be related to the fix provided by #502?

Anybody can try whether it will help t get rid of failure. But some of comments 
says that all failures are not fixed ... 
https://github.com/SSSD/sssd/pull/502#issuecomment-360409977
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/496#issuecomment-361393361
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#497][comment] sysdb: sanitize search filter input - backport sssd-1-14

2018-01-29 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/497
Title: #497: sysdb: sanitize search filter input - backport sssd-1-14

fidencio commented:
"""
Both the patch and the test that is together work as expected!
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/497#issuecomment-361226965
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][+Accepted] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

fidencio commented:
"""
CI has passed apart from some issue running mock-build on rawhide: 
http://vm-031.${abc}/logs/job/83/25/summary.html

May the failure there be related to the fix provided by 
https://github.com/SSSD/sssd/pull/502?
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/496#issuecomment-361212750
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#496][comment] sysdb: sanitize search filter input - backport sssd-1-13

2018-01-29 Thread fidencio
  URL: https://github.com/SSSD/sssd/pull/496
Title: #496: sysdb: sanitize search filter input - backport sssd-1-13

fidencio commented:
"""
Both the patch and the test that is together work as expected!
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/496#issuecomment-361210174
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org


[SSSD] [sssd PR#485][comment] Support alternative sources for the files provider

2018-01-29 Thread pbrezina
  URL: https://github.com/SSSD/sssd/pull/485
Title: #485:  Support alternative sources for the files provider 

pbrezina commented:
"""
No, that would not be sufficient. You want to put one transaction around 
deleting and storing users/groups to ensure that nothing is deleted if store 
failes, i.e. around the code a quoted.
```
start-transaction()
delete-users()
sf_enum_users()
commit/cancel-transaction()
```
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/485#issuecomment-361167080
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org