sssd-devel July 2020

sssd-devel@lists.fedorahosted.org

3 participants
91 discussions

Start a nNew thread

[sssd PR#5228][-Waiting for review] Test: Add back removed tests
by elkoniu 19 Jul '20

19 Jul '20

URL: https://github.com/SSSD/sssd/pull/5228 Title: #5228: Test: Add back removed tests Label: -Waiting for review

1 0

[sssd PR#5223][comment] [wip] Add .clang-format
by pbrezina 17 Jul '20

17 Jul '20

URL: https://github.com/SSSD/sssd/pull/5223 Title: #5223: [wip] Add .clang-format pbrezina commented: """ I pushed little update. So far it works great for development. But it has some issues when trying to automatically correct the coding style in some scenarios. It tries to change more code then it is supposed to when git diff creates bigger snippet then a small change self-contained change. """ See the full comment at https://github.com/SSSD/sssd/pull/5223#issuecomment-660077063

1 0

[sssd PR#5223][synchronized] [wip] Add .clang-format
by pbrezina 17 Jul '20

17 Jul '20

URL: https://github.com/SSSD/sssd/pull/5223 Author: pbrezina Title: #5223: [wip] Add .clang-format Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5223/head:pr5223 git checkout pr5223

1 0

[sssd PR#5244][+Ready to push] sssctl: sssctl config-check alternative snippet dir
by pbrezina 17 Jul '20

17 Jul '20

URL: https://github.com/SSSD/sssd/pull/5244 Title: #5244: sssctl: sssctl config-check alternative snippet dir Label: +Ready to push

1 0

[sssd PR#5228][edited] Test: Add back removed tests
by ikerexxe 17 Jul '20

17 Jul '20

URL: https://github.com/SSSD/sssd/pull/5228 Author: ikerexxe Title: #5228: Test: Add back removed tests Action: edited Changed field: body Original value: """ Tests: ```users_by_filter_multiple_domains_valid``` and ```groups_by_filter_multiple_domains_valid``` were removed in the past and these commits add them back. ```users_by_filter_multiple_domains_valid``` test explanation: Given two users are present When the users are searched by filtering domains Then the two users are returned correctly. ```groups_by_filter_multiple_domains_valid``` test explanation: Given two groups are present When the groups are searched by filtering domains Then the two groups are returned correctly. Resolves: https://github.com/SSSD/sssd/issues/3920 """

1 0

[sssd PR#5237][comment] sss_ssh_knownhostsproxy: print error when unable to connect
by pbrezina 16 Jul '20

16 Jul '20

URL: https://github.com/SSSD/sssd/pull/5237 Title: #5237: sss_ssh_knownhostsproxy: print error when unable to connect pbrezina commented: """ Thank you. See the new patch. """ See the full comment at https://github.com/SSSD/sssd/pull/5237#issuecomment-659668101

1 0

[sssd PR#5237][synchronized] sss_ssh_knownhostsproxy: print error when unable to connect
by pbrezina 16 Jul '20

16 Jul '20

URL: https://github.com/SSSD/sssd/pull/5237 Author: pbrezina Title: #5237: sss_ssh_knownhostsproxy: print error when unable to connect Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5237/head:pr5237 git checkout pr5237

1 0

[sssd PR#1036][comment] certmap: sanitize LDAP search filter
by jsf9k 16 Jul '20

16 Jul '20

URL: https://github.com/SSSD/sssd/pull/1036 Title: #1036: certmap: sanitize LDAP search filter jsf9k commented: """ I am running into this same issue, also having to do with parentheses in CNs when dealing with U.S. government smart cards. I'm going to have to configure FreeIPA to match on full certificates for now and revert back to using certmap data once this PR is merged. """ See the full comment at https://github.com/SSSD/sssd/pull/1036#issuecomment-659416781

1 0

[sssd PR#5246][comment] (WiP) Drop support of libnss as a crypto backend
by alexey-tikhonov 15 Jul '20

15 Jul '20

URL: https://github.com/SSSD/sssd/pull/5246 Title: #5246: (WiP) Drop support of libnss as a crypto backend alexey-tikhonov commented: """ PR is marked as "WiP" because I plan to add another patch to rename misleading `--nssdb` arg of `p11_child`. Still I posted it so that anyone interested could have a preliminary look. """ See the full comment at https://github.com/SSSD/sssd/pull/5246#issuecomment-658908989

1 0

[sssd PR#5213][comment] config: [RFE] Add "enabled" option to domain section
by alexey-tikhonov 15 Jul '20

15 Jul '20

URL: https://github.com/SSSD/sssd/pull/5213 Title: #5213: config: [RFE] Add "enabled" option to domain section alexey-tikhonov commented: """ > Pushed PR: #5213 > > * `master` > > * [ff8d7b8](https://github.com/SSSD/sssd/commit/ff8d7b8f0dca57d04ed4157bc60ad3… - config: [RFE] Add "enabled" option to domain section This patch introduced following compilation warning: ``` ../src/confdb/confdb.c: In function ‘confdb_get_domains’: ../src/confdb/confdb.c:2499:12: warning: ‘enabled’ may be used uninitialized in this function [-Wmaybe-uninitialized] 2499 | if (enabled && !string_in_list(all_domains[idx], domlist, false)) { | ^ ../src/confdb/confdb.c:2460:10: note: ‘enabled’ was declared here 2460 | bool enabled; | ^~~~~~~ ``` This is clearly false positive, but still would be good to fix. """ See the full comment at https://github.com/SSSD/sssd/pull/5213#issuecomment-658725445

1 0

← Newer
1
2
3
4
5
6
7
8
9
10
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel July 2020