[SSSD] SSSD: fix for CVE-2023-3758
Hi SSSD maintainers, a fix for CVE-2023-3758 was made available upstream via https://github.com/SSSD/sssd/pull/7302#issuecomment-2063480554 -- ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[SSSD] Re: 2.9.{2,4} test failure on armhf: responder_cache_req-tests and pam-srv-tests
Btw, about a patch https://code.launchpad.net/~ahasenack/ubuntu/+source/sssd/+git/sssd/+merge/463721/+preview-diff/1051133/+files/preview.diff that I've found browsing links list: wouldn't it be better to use `SPRItime` (defined in 'util/sss_format.h') instead of replacing `%lu` with `%lld`? And I think you could upstream this. On Fri, Apr 5, 2024 at 12:24 PM Alexey Tikhonov wrote: > > On Thu, Apr 4, 2024 at 6:26 PM Andreas Hasenack wrote: > > > > Hi, > > > > I'm troubleshooting a set of test failures[1][2] in sssd 2.9.2 and > > 2.9.4. They are only failing in armhf, so the usual culprits are > > 32bits, slowness of the arch/infrastructure, and now we also have a > > time_t change[3] to a 64bit type. > > + unaligned mem access, but hardly the case here. > > > I don't know yet which, if any, are > > related to the failures. > > > > The current failures are: > > > > [ RUN ] test_user_by_recent_filter_valid > > [ ERROR ] --- 0x2 != 0x1 > > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2505: > > error: Failure! > > [ FAILED ] test_user_by_recent_filter_valid > > [ RUN ] test_users_by_recent_filter_valid > > [ OK ] test_users_by_recent_filter_valid > > [ RUN ] test_group_by_recent_filter_valid > > [ ERROR ] --- 0x2 != 0x1 > > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2868: > > error: Failure! > > [ FAILED ] test_group_by_recent_filter_valid > > [ RUN ] test_groups_by_recent_filter_valid > > [ OK ] test_groups_by_recent_filter_valid > > [ RUN ] test_users_by_filter_filter_old > > [ ERROR ] --- 0x2 != 0x1 > > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2613: > > error: Failure! > > [ FAILED ] test_users_by_filter_filter_old > > > > And > > > > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > > value to mock function sss_dp_get_account_recv > > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > > value was declared here > > > > [ FAILED ] test_pam_preauth_expired_crl_file > > ... > > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > > value to mock function sss_dp_get_account_recv > > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > > value was declared here > > > > [ FAILED ] test_pam_preauth_crl_another_ca_crl_invalid_files > > [ RUN ] test_pam_preauth_crl_invalid_crl_another_ca_files > > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > > value to mock function sss_dp_get_account_recv > > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > > value was declared here > > > > [ FAILED ] test_pam_preauth_crl_invalid_crl_another_ca_files > > > > At first, I've been trying to understand > > test_user_by_recent_filter_valid()[4], and it seems to be the reason > > for all 3 failures in that test group. > > > > The failure is that it expects just one user to be returned, but in > > the end there are two. > > I would print the content of `test_ctx->result` at this point just to > debug / see what's there... > > > The test seems to prepare two users, > > Why? > ``` > test_ctx->create_user1 = true; > test_ctx->create_user2 = false; > ``` > > > but in the > > end only one should be returned by the filter. I have not yet > > understood what is being expected here. What's "recent" in this > > context? Is it related to the "one second ago" (time(NULL) -1) in > > prepare_user()? > > > > > > > > 1. https://bugs.launchpad.net/ubuntu/+source/sssd/+bug/2058576 > > 2. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1068063 > > 3. https://wiki.debian.org/ReleaseGoals/64bit-time > > 4. > > https://github.com/SSSD/sssd/blob/1c2aa825062dcf2da2e886c3211be90c22db1750/src/tests/cmocka/test_responder_cache_req.c#L2467 > > -- > > ___ > > sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org > > To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org > > Fedora Code of Conduct: > > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > > List Guidelines: https://fedoraproject.org/
[SSSD] Re: 2.9.{2,4} test failure on armhf: responder_cache_req-tests and pam-srv-tests
On Thu, Apr 4, 2024 at 6:26 PM Andreas Hasenack wrote: > > Hi, > > I'm troubleshooting a set of test failures[1][2] in sssd 2.9.2 and > 2.9.4. They are only failing in armhf, so the usual culprits are > 32bits, slowness of the arch/infrastructure, and now we also have a > time_t change[3] to a 64bit type. + unaligned mem access, but hardly the case here. > I don't know yet which, if any, are > related to the failures. > > The current failures are: > > [ RUN ] test_user_by_recent_filter_valid > [ ERROR ] --- 0x2 != 0x1 > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2505: > error: Failure! > [ FAILED ] test_user_by_recent_filter_valid > [ RUN ] test_users_by_recent_filter_valid > [ OK ] test_users_by_recent_filter_valid > [ RUN ] test_group_by_recent_filter_valid > [ ERROR ] --- 0x2 != 0x1 > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2868: > error: Failure! > [ FAILED ] test_group_by_recent_filter_valid > [ RUN ] test_groups_by_recent_filter_valid > [ OK ] test_groups_by_recent_filter_valid > [ RUN ] test_users_by_filter_filter_old > [ ERROR ] --- 0x2 != 0x1 > [ LINE ] --- ../src/tests/cmocka/test_responder_cache_req.c:2613: > error: Failure! > [ FAILED ] test_users_by_filter_filter_old > > And > > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > value to mock function sss_dp_get_account_recv > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > value was declared here > > [ FAILED ] test_pam_preauth_expired_crl_file > ... > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > value to mock function sss_dp_get_account_recv > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > value was declared here > > [ FAILED ] test_pam_preauth_crl_another_ca_crl_invalid_files > [ RUN ] test_pam_preauth_crl_invalid_crl_another_ca_files > [ ERROR ] --- No entries for symbol sss_dp_get_account_recv. > ../src/tests/cmocka/common_mock_resp_dp.c:52: error: Could not get > value to mock function sss_dp_get_account_recv > ../src/tests/cmocka/test_pam_srv.c:802: note: Previously returned mock > value was declared here > > [ FAILED ] test_pam_preauth_crl_invalid_crl_another_ca_files > > At first, I've been trying to understand > test_user_by_recent_filter_valid()[4], and it seems to be the reason > for all 3 failures in that test group. > > The failure is that it expects just one user to be returned, but in > the end there are two. I would print the content of `test_ctx->result` at this point just to debug / see what's there... > The test seems to prepare two users, Why? ``` test_ctx->create_user1 = true; test_ctx->create_user2 = false; ``` > but in the > end only one should be returned by the filter. I have not yet > understood what is being expected here. What's "recent" in this > context? Is it related to the "one second ago" (time(NULL) -1) in > prepare_user()? > > > > 1. https://bugs.launchpad.net/ubuntu/+source/sssd/+bug/2058576 > 2. https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1068063 > 3. https://wiki.debian.org/ReleaseGoals/64bit-time > 4. > https://github.com/SSSD/sssd/blob/1c2aa825062dcf2da2e886c3211be90c22db1750/src/tests/cmocka/test_responder_cache_req.c#L2467 > -- > ___ > sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org > To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org > Do not reply to spam, report it: > https://pagure.io/fedora-infrastructure/new_issue -- ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[SSSD] Re: Why does systemd implement its own wire protocol?
Hello, On Tue, Aug 2, 2022 at 8:46 PM Dominik George wrote: > Hi, > > during my evaluations and researches around implementing user and > device management using OAuth and OIDC, I also caught up on the > details of: > > * sssd's architecture [1] > * Freedesktop's AccountsService [2] > * the idea making sssd implement AccountsService [3] > > Currently, I am evaluating whether I should add an OIDC module to > sssd, or write my own authentication daemon handling it (mimicing sssd > to some extent). I will probably get back to the pros and cons around > this later in a separate thread. > > One question though that I did not find an answer to, which is > somewhat important for both approaches, is: > > Why does sssd implement its own wire protocol between client > libraries and responders, instead of using the D-Bus system bus? > I'd say, (one of the) main reason(s): to minimize a set of dependencies pulled into random applications using SSSD client libraries (often implicitly, thru, for example, glibc). Security considerations are also important, especially when we talk about PAM and system bus. There are, perhaps, other reasons, but I think even those two are enough. > > Actually, designing a draft for my own authentication daemon, I > decided to go with my own wire protocol (based on protobuf) as well, > and then thought that if I was going to implement a realmd provider > later on, I'd have to talk D-Bus anyway, so I could just as well use > it everywhere. > > Since sssd decided to not do that, I assume there is a reason, so: > > If I were designing a system like sssd, why should I not use D-Bus > to communicate between my PAM module and my backend daemon? > > Disclaimer: I am not seeking to replace sssd; right now, all of this > is of purely academic nature, to understand the whole picture. > > Cheers, > Nik > > [1] https://sssd.io/docs/architecture.html > [2] https://www.freedesktop.org/wiki/Software/AccountsService/ > [3] https://docs.pagure.org/sssd.sssd/design_pages/accounts_service.html > ___ > sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org > To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org > Do not reply to spam on the list, report it: > https://pagure.io/fedora-infrastructure > ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[SSSD] Re: Documentation on how to write backends?
Hi, On Sun, Jul 31, 2022 at 5:42 PM Dominik George wrote: > Hi, > > I am currently working on integrating Linux desktops directly with > web-based login services, using OAuth/OpenID Connect. > Perhaps makes sense to take a look if you can reuse https://github.com/SSSD/sssd/commit/878737c8ee25b40179516a302378c7cabaf4487d > > For now, I have written my own NSS and PAM modules, but it would > be better to decouple the time-consuming work of querying the data > source. > > I was wondering whether third-party backends can be written for > sssd? I cannot find documentation on that. > > Cheers and thank you, > Nik > ___ > sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org > To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org > Fedora Code of Conduct: > https://docs.fedoraproject.org/en-US/project/code-of-conduct/ > List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines > List Archives: > https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org > Do not reply to spam on the list, report it: > https://pagure.io/fedora-infrastructure > ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] Re: sssd linking with krb5's libkrad0
Hi,
On Mon, Jun 20, 2022 at 5:19 PM Andreas Hasenack wrote:
>
> Hi,
>
> I was reviewing an Ubuntu merge of sssd 2.7.1
As a side note: it's better to skip 2.7.1 altogether as it has a
serious regression and to use sssd-2.7.2
> and saw that the new
> sssd-idp package links with MIT kerberos libkrad0. libkrad-dev in
> Debian has this notice in its description[1]:
> ```
> This package includes development headers for libkrad0, the MIT
> Kerberos RADIUS library. You should not use this RADIUS library in
> packages unrelated to MIT Kerberos.
> ```
JFTR, 'sssd-idp' ships a new Kerberos plugin
('sssd_krb5_idp_plugin.so') so in this sense "related" to MIT
Kerberos.
>
> It looks like that was added as a result of dealing with Debian bug
> #735323[1], which is what introduced the libkrad-dev package in
> Debian.
>
> Currently in Ubuntu (and likely Debian, I haven't checked) libkrad0
> has only krb5 itself has a reverse dependency[3], and this change will
> make sssd a new rdep.
>
> Is that notice in libkrad-dev's package description still current, and
> external projects shouldn't link with libkrad0? What's the reasoning?
> Or is that no longer relevant?
>
> I'm CCing package maintainers who might not be subscribed to
> sssd-devel, please include them in any replies if you can.
>
>
> 1. https://salsa.debian.org/debian/krb5/-/blob/master/debian/control#L471
> 2. https://www.debian.org/Bugs/#735323
> 3.
> https://people.canonical.com/~ubuntu-archive/germinate-output/ubuntu.jammy/rdepends/krb5/libkrad0
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure
[SSSD] [ding-libs] Announcing ding-libs 0.6.2
# ding-libs 0.6.2 The SSSD team announces a minor maintenance update of ding-libs - version 0.6.2 The tarball can be downloaded from: https://github.com/SSSD/ding-libs/releases/tag/0.6.2 Tickets fixed: - https://pagure.io/SSSD/ding-libs/issue/3182 - https://github.com/SSSD/ding-libs/issues/5 - https://github.com/SSSD/ding-libs/issues/7 - https://github.com/SSSD/ding-libs/issues/8 - https://github.com/SSSD/ding-libs/issues/9 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] Re: Heads up. Moving 'ding-libs' to github. (Date to be defined)
Hello, aforementioned migration of the 'ding-libs' project to Github was completed: now the only used location is https://github.com/SSSD/ding-libs On Tue, Nov 9, 2021 at 8:56 PM Alexey Tikhonov wrote: > > Hello, > > as a follow up to the complete SSSD migration to github [1], we > decided to continue with the consolidation effort and to make the same > step with the 'ding-libs' project that currently uses both Pagure [2] > and Github [3]. > > This is only a heads up. Precise date isn't set yet. > > But as a first step: we will no longer accept new bug reports at > Pagure and we will kindly ask you to use Github instead. > > Thank you, > Alexey. > > > [1] https://lists.fedoraproject.org/archives/list/sssd-us...@lists.fedorahosted.org/thread/PKOGRIHT5DCYTEEHN7CSRGHJZIQFWEUI/ > [2] https://pagure.io/SSSD/ding-libs > [3] https://github.com/SSSD/ding-libs ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5949][comment] krb5: AD and IPA don't change Kerberos port
URL: https://github.com/SSSD/sssd/pull/5949 Title: #5949: krb5: AD and IPA don't change Kerberos port alexey-tikhonov commented: """ Hi @ikerexxe, Wouldn't it make sense to add a RN to commit to stress a 2.6.2 regression was fixed? """ See the full comment at https://github.com/SSSD/sssd/pull/5949#issuecomment-1014422652 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5927][closed] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Author: sumit-bose Title: #5927: ad: add required 'cn' attribute to subdomain object Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5927/head:pr5927 git checkout pr5927 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5927][-Ready to push] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5927][-Accepted] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5927][+Pushed] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5927][comment] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5927 * `master` * bf6059eb55c8caa3111ef718db1676c96a67c084 - ad: add required 'cn' attribute to subdomain object * `sssd-1-16` * 64192cf7c2823ae93820623b0ae285b697fabe12 - ad: add required 'cn' attribute to subdomain object """ See the full comment at https://github.com/SSSD/sssd/pull/5927#issuecomment-1014418262 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][comment] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails alexey-tikhonov commented: """ Hi @scabrero, what are your plans wrt this PR? """ See the full comment at https://github.com/SSSD/sssd/pull/5881#issuecomment-1013849967 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5674][-Waiting for review] [WIP] This is just a test
URL: https://github.com/SSSD/sssd/pull/5674 Title: #5674: [WIP] This is just a test Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][comment] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 alexey-tikhonov commented: """ Pavel mentioned that possible reason for the check might be https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ldap/searching-using-range-retrieval """ See the full comment at https://github.com/SSSD/sssd/pull/5937#issuecomment-1012077810 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5925][comment] TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode
URL: https://github.com/SSSD/sssd/pull/5925 Title: #5925: TEST: Current value of ssh_hash_known_hosts causes error in the default configuration in FIPS mode alexey-tikhonov commented: """ And it still fails `pycodestyle`: ``` ./src/tests/multihost/ipa/test_misc.py:388:58: W292 no newline at end of file ``` """ See the full comment at https://github.com/SSSD/sssd/pull/5925#issuecomment-1009127835 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5929][comment] Tests: Add tests for poor man's backtrace
URL: https://github.com/SSSD/sssd/pull/5929 Title: #5929: Tests: Add tests for poor man's backtrace alexey-tikhonov commented: """ Hi, I have a couple of questions, but in general looks good to me. """ See the full comment at https://github.com/SSSD/sssd/pull/5929#issuecomment-1007764283 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][comment] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 alexey-tikhonov commented: """ I did actual testing and conclusions are: - I correctly identified the bottleneck: https://github.com/SSSD/sssd/blob/4897c28741112b547a69feb7c887764c64cc9540/src/providers/ldap/sdap_async_nested_groups.c#L2355 - my patch(-es) doesn't resolve the issue because (in certain (or most?) configs) by the moment `Received 4 dereference results` `members->num_values` is already 4 (i.e. all those members are already known) """ See the full comment at https://github.com/SSSD/sssd/pull/5937#issuecomment-1006932032 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][comment] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 alexey-tikhonov commented: """ I did actual testing and conclusions are: - I correctly identified the bottleneck: https://github.com/SSSD/sssd/blob/4897c28741112b547a69feb7c887764c64cc9540/src/providers/ldap/sdap_async_nested_groups.c#L2355 - my patch(-es) doesn't resolve the issue because (in certain configs) by the moment `Received 4 dereference results` `members->num_values` is already 4 (i.e. all those members are already known) """ See the full comment at https://github.com/SSSD/sssd/pull/5937#issuecomment-1006932032 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][comment] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 alexey-tikhonov commented: """ I did actual testing and conclusions are: - I correctly identified the bottleneck: https://github.com/SSSD/sssd/blob/4897c28741112b547a69feb7c887764c64cc9540/src/providers/ldap/sdap_async_nested_groups.c#L2355 - my patch(-es) doesn't resolve the issue because by the moment `Received 4 dereference results` `members->num_values` is already 4 (i.e. all those members are already known) """ See the full comment at https://github.com/SSSD/sssd/pull/5937#issuecomment-1006932032 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][closed] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Author: jakub-vavra-cz Title: #5892: Tests: Add a test for BZ2004406 Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5892/head:pr5892 git checkout pr5892 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][-Ready to push] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][-Accepted] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][+Pushed] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][comment] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5892 * `master` * 4897c28741112b547a69feb7c887764c64cc9540 - Tests: Add a test for BZ2004406 """ See the full comment at https://github.com/SSSD/sssd/pull/5892#issuecomment-1006508391 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][-Ready to push] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][closed] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Author: shridhargadekar Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5930/head:pr5930 git checkout pr5930 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][-Accepted] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][+Pushed] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][comment] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5930 * `master` * 58b3233f06c796c0928a80da63fe01a1c1fa05a4 - Tests: Health and Support Analyzer - Add request log parsing utility """ See the full comment at https://github.com/SSSD/sssd/pull/5930#issuecomment-1006508020 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][closed] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Author: aborah-sudo Title: #5942: Tests: Fix python-alltests-tier1-2 Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5942/head:pr5942 git checkout pr5942 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][-Accepted] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Title: #5942: Tests: Fix python-alltests-tier1-2 Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][-Ready to push] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Title: #5942: Tests: Fix python-alltests-tier1-2 Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][+Pushed] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Title: #5942: Tests: Fix python-alltests-tier1-2 Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][comment] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Title: #5942: Tests: Fix python-alltests-tier1-2 alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5942 * `master` * 9ba593e9a1639377a89fc06cfd30b65d85517612 - Tests: Fix python-alltests-tier1-2 """ See the full comment at https://github.com/SSSD/sssd/pull/5942#issuecomment-1006507739 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][+Ready to push] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5930][+Ready to push] Tests: Health and Support Analyzer - Add request log parsing utility
URL: https://github.com/SSSD/sssd/pull/5930 Title: #5930: Tests: Health and Support Analyzer - Add request log parsing utility Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5942][+Ready to push] Tests: Fix python-alltests-tier1-2
URL: https://github.com/SSSD/sssd/pull/5942 Title: #5942: Tests: Fix python-alltests-tier1-2 Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD]WARNING: MERGED [sssd PR#5932][closed] Translations update from Fedora Weblate
*WARNING: this pull request has been merged!* This is only mirrored repo thus any changes will be erased. Please push commit(s) to authoritative repository. URL: https://github.com/SSSD/sssd/pull/5932 Author: weblate Title: #5932: Translations update from Fedora Weblate Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5932/head:pr5932 git checkout pr5932 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][closed] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Author: alexey-tikhonov Title: #5936: RESPONDER: reduce log level in case inconsistent Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5936/head:pr5936 git checkout pr5936 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][-Ready to push] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][-Accepted] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][+Pushed] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][comment] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5936 * `master` * 868f38742e0c8082a392814cd2b3e637c1da3c17 - RESPONDER: reduce log level in case files provider in inconsistent state falls back to NSS. """ See the full comment at https://github.com/SSSD/sssd/pull/5936#issuecomment-1005859521 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][+Ready to push] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][closed] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Author: alexey-tikhonov Title: #5934: Removed unused file. Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5934/head:pr5934 git checkout pr5934 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][-Ready to push] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][-Accepted] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][+Pushed] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][comment] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5934 * `master` * 28af1752a8ba4d81ec3c6046d597f2c19d4d4755 - Removed unused file. """ See the full comment at https://github.com/SSSD/sssd/pull/5934#issuecomment-1005844328 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][closed] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Author: alexey-tikhonov Title: #5935: sysdb: more specific mpg search filter Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5935/head:pr5935 git checkout pr5935 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][-Ready to push] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][-Accepted] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][comment] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5935 * `sssd-1-16` * 0dde096aac1db7d8533f51254f6a8c22a67114d8 - sysdb: more specific mpg search filter """ See the full comment at https://github.com/SSSD/sssd/pull/5935#issuecomment-1005844004 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][+Pushed] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][closed] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Author: alexey-tikhonov Title: #5941: Minor updates in `IPA: get_object_from_cache()` Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5941/head:pr5941 git checkout pr5941 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][-Ready to push] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][-Accepted] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][+Pushed] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][comment] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5941 * `master` * e9a25bb0b07130e49a50254333718753f95e1602 - IPA: get_object_from_cache(): - reduce log level in case object wasn't found in cache - slightly reduce code duplication * 104f513c46a21753faae890b9f753ba26bb21aa6 - IPA: get_object_from_cache(): don't touch output arg `_msg` in case object wasn't found (i.e. ENOENT returned) """ See the full comment at https://github.com/SSSD/sssd/pull/5941#issuecomment-1005843499 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][+Ready to push] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][+Ready to push] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][+Ready to push] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][-Waiting for review] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][+Changes requested] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][comment] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails alexey-tikhonov commented: """ One of integration tests is broken: ``` test_group_operations _ Traceback (most recent call last): File "/shared/sssd/src/tests/intg/test_pysss_nss_idmap.py", line 237, in test_group_operations group_id = grp.getgrnam(group).gr_gid KeyError: "getgrnam(): name not found: 'group3_dom1-17775'" === short test summary info FAILED test_pysss_nss_idmap.py::test_group_operations - KeyError: "getgrnam()... ``` """ See the full comment at https://github.com/SSSD/sssd/pull/5881#issuecomment-1005095008 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][+Waiting for review] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941 Title: #5941: Minor updates in `IPA: get_object_from_cache()` Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5941][synchronized] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941
Author: alexey-tikhonov
Title: #5941: Minor updates in `IPA: get_object_from_cache()`
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5941/head:pr5941
git checkout pr5941
From ab69a01fc93ec81753ed771b1886e87e674c3280 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 4 Jan 2022 19:22:51 +0100
Subject: [PATCH 1/2] IPA: get_object_from_cache(): don't touch output arg
`_msg` in case object wasn't found (i.e. ENOENT returned)
---
src/providers/ipa/ipa_subdomains_id.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c
index 445b9ba2ff..901382084a 100644
--- a/src/providers/ipa/ipa_subdomains_id.c
+++ b/src/providers/ipa/ipa_subdomains_id.c
@@ -1260,10 +1260,14 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
goto done;
}
-if (ret != EOK && ret != ENOENT) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
+if (ret != EOK) {
+if (ret != ENOENT) {
+DEBUG(SSSDBG_OP_FAILURE,
+ "Failed to make request to our cache: [%d]: [%s]\n",
+ ret, sss_strerror(ret));
+} else {
+DEBUG(SSSDBG_FUNC_DATA, "Object wasn't found in cache");
+}
goto done;
}
From 704a7cc4bb83f600bd45973dbdafb2e89c35c635 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 4 Jan 2022 19:48:43 +0100
Subject: [PATCH 2/2] IPA: get_object_from_cache(): - reduce log level in case
object wasn't found in cache - slightly reduce code duplication
---
src/providers/ipa/ipa_subdomains_id.c | 54 +--
1 file changed, 17 insertions(+), 37 deletions(-)
diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c
index 901382084a..ccf41db600 100644
--- a/src/providers/ipa/ipa_subdomains_id.c
+++ b/src/providers/ipa/ipa_subdomains_id.c
@@ -1138,50 +1138,29 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
if (ar->filter_type == BE_FILTER_SECID) {
ret = sysdb_search_object_by_sid(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
+if (ret == EOK) {
+*_msg = res->msgs[0];
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_UUID) {
ret = sysdb_search_object_by_uuid(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
+if (ret == EOK) {
+*_msg = res->msgs[0];
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_CERT) {
ret = sysdb_search_object_by_cert(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
-}
-if (res->count != 1) {
-DEBUG(SSSDBG_OP_FAILURE,
- "More than one result found in our cache\n");
-ret = EINVAL;
-goto done;
+if (ret == EOK) {
+if (res->count != 1) {
+DEBUG(SSSDBG_OP_FAILURE,
+ "More than one result found in our cache\n");
+ret = EINVAL;
+} else {
+*_msg = res->msgs[0];
+}
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_IDNUM) {
id = strtouint32(ar->filter_value, &endptr, 10);
@@ -1260,6 +1239,11 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
goto done;
}
+if (ret == EOK) {
+*_msg = msg;
+}
+
+done:
if (ret != EOK) {
if (ret != ENOENT) {
DEBUG(SSSDBG_OP_FAILURE,
@@ -1268,12 +1252,8 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
} else {
DEBUG(SSSDBG_FUNC_DATA, "Object wasn't found in cache");
}
-goto done;
}
-*_
[SSSD] [sssd PR#5941][opened] Minor updates in `IPA: get_object_from_cache()`
URL: https://github.com/SSSD/sssd/pull/5941
Author: alexey-tikhonov
Title: #5941: Minor updates in `IPA: get_object_from_cache()`
Action: opened
PR body:
"""
None
"""
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5941/head:pr5941
git checkout pr5941
From ab69a01fc93ec81753ed771b1886e87e674c3280 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 4 Jan 2022 19:22:51 +0100
Subject: [PATCH 1/2] IPA: get_object_from_cache(): don't touch output arg
`_msg` in case object wasn't found (i.e. ENOENT returned)
---
src/providers/ipa/ipa_subdomains_id.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c
index 445b9ba2ff..901382084a 100644
--- a/src/providers/ipa/ipa_subdomains_id.c
+++ b/src/providers/ipa/ipa_subdomains_id.c
@@ -1260,10 +1260,14 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
goto done;
}
-if (ret != EOK && ret != ENOENT) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
+if (ret != EOK) {
+if (ret != ENOENT) {
+DEBUG(SSSDBG_OP_FAILURE,
+ "Failed to make request to our cache: [%d]: [%s]\n",
+ ret, sss_strerror(ret));
+} else {
+DEBUG(SSSDBG_FUNC_DATA, "Object wasn't found in cache");
+}
goto done;
}
From 6fbf2dd9405efa73483f4b9774513aaede49af59 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 4 Jan 2022 19:48:43 +0100
Subject: [PATCH 2/2] IPA: get_object_from_cache(): - reduce log level in case
object wasn't found in cache - slightly reduce code duplication
---
src/providers/ipa/ipa_subdomains_id.c | 53 +--
1 file changed, 17 insertions(+), 36 deletions(-)
diff --git a/src/providers/ipa/ipa_subdomains_id.c b/src/providers/ipa/ipa_subdomains_id.c
index 901382084a..28712272d6 100644
--- a/src/providers/ipa/ipa_subdomains_id.c
+++ b/src/providers/ipa/ipa_subdomains_id.c
@@ -1138,50 +1138,29 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
if (ar->filter_type == BE_FILTER_SECID) {
ret = sysdb_search_object_by_sid(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
+if (ret == EOK) {
+*_msg = res->msgs[0];
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_UUID) {
ret = sysdb_search_object_by_uuid(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
+if (ret == EOK) {
+*_msg = res->msgs[0];
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_CERT) {
ret = sysdb_search_object_by_cert(mem_ctx, dom, ar->filter_value, attrs,
&res);
-if (ret != EOK) {
-DEBUG(SSSDBG_OP_FAILURE,
- "Failed to make request to our cache: [%d]: [%s]\n",
- ret, sss_strerror(ret));
-goto done;
-}
-if (res->count != 1) {
-DEBUG(SSSDBG_OP_FAILURE,
- "More than one result found in our cache\n");
-ret = EINVAL;
-goto done;
+if (ret == EOK) {
+if (res->count != 1) {
+DEBUG(SSSDBG_OP_FAILURE,
+ "More than one result found in our cache\n");
+ret = EINVAL;
+} else {
+*_msg = res->msgs[0];
+}
}
-
-*_msg = res->msgs[0];
-
-ret = EOK;
goto done;
} else if (ar->filter_type == BE_FILTER_IDNUM) {
id = strtouint32(ar->filter_value, &endptr, 10);
@@ -1260,6 +1239,11 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
goto done;
}
+if (ret == EOK) {
+*_msg = msg;
+}
+
+done:
if (ret != EOK) {
if (ret != ENOENT) {
DEBUG(SSSDBG_OP_FAILURE,
@@ -1271,9 +1255,6 @@ errno_t get_object_from_cache(TALLOC_CTX *mem_ctx,
goto done;
}
-*_msg = msg;
-
-done:
return ret;
}
_
[SSSD] [sssd PR#5927][+Waiting for review] ad: add required 'cn' attribute to subdomain object
URL: https://github.com/SSSD/sssd/pull/5927 Title: #5927: ad: add required 'cn' attribute to subdomain object Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5908][comment] Check default debug level of sssd and corresponding logs
URL: https://github.com/SSSD/sssd/pull/5908 Title: #5908: Check default debug level of sssd and corresponding logs alexey-tikhonov commented: """ > Thank you for the updates. Looks good to me. @jakub-vavra-cz, could you please take a look as well? """ See the full comment at https://github.com/SSSD/sssd/pull/5908#issuecomment-1005047491 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][+Changes requested] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5892][-Waiting for review] Tests: Add a test for BZ2004406
URL: https://github.com/SSSD/sssd/pull/5892 Title: #5892: Tests: Add a test for BZ2004406 Label: -Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][comment] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails alexey-tikhonov commented: """ I think we are done with 2.6.z series and can proceed with this PR. """ See the full comment at https://github.com/SSSD/sssd/pull/5881#issuecomment-1005046508 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][+Waiting for review] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5881][-Changes requested] SDAP: Do not fail ASQ search when parsing a referenced entry fails
URL: https://github.com/SSSD/sssd/pull/5881 Title: #5881: SDAP: Do not fail ASQ search when parsing a referenced entry fails Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5830][+Waiting for review] Tests: Port the old ns_account_lock.sh script to pytest
URL: https://github.com/SSSD/sssd/pull/5830 Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5786][comment] Tests: [SSSD-3579]: Skip test test_0018_bz1734040.
URL: https://github.com/SSSD/sssd/pull/5786 Title: #5786: Tests: [SSSD-3579]: Skip test test_0018_bz1734040. alexey-tikhonov commented: """ @jakub-vavra-cz, @sgoveas, what is the status of this PR? """ See the full comment at https://github.com/SSSD/sssd/pull/5786#issuecomment-1005041046 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][-Accepted] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][closed] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Author: sumit-bose Title: #5940: ipa: fix reply socket of selinux_child Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5940/head:pr5940 git checkout pr5940 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][-Ready to push] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][+Pushed] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][comment] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child alexey-tikhonov commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5940 * `master` * 5a2e0ebe83913e317f66478daeff35987c278e27 - ipa: fix reply socket of selinux_child """ See the full comment at https://github.com/SSSD/sssd/pull/5940#issuecomment-1004751039 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][+Ready to push] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][+Accepted] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5940][comment] ipa: fix reply socket of selinux_child
URL: https://github.com/SSSD/sssd/pull/5940 Title: #5940: ipa: fix reply socket of selinux_child alexey-tikhonov commented: """ Thank you, ACK. """ See the full comment at https://github.com/SSSD/sssd/pull/5940#issuecomment-1004644271 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][comment] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent alexey-tikhonov commented: """ > I think commit message of this change is a bit unclear. Please take a look at the updated version. """ See the full comment at https://github.com/SSSD/sssd/pull/5936#issuecomment-1004117129 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][+Waiting for review] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][synchronized] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936
Author: alexey-tikhonov
Title: #5936: RESPONDER: reduce log level in case inconsistent
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5936/head:pr5936
git checkout pr5936
From 4b524be7efa46b70946c7f2cda0fb57a7f4e7b8d Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 28 Dec 2021 16:30:43 +0100
Subject: [PATCH] RESPONDER: reduce log level in case files provider in
inconsistent state falls back to NSS.
`ENOENT` return code of `sss_dp_account_files_params()` means
"can't serve request, but 'falls back to NSS' is enabled".
This (consciously configured) scenario doesn't justify `SSSDBG_OP_FAILURE`
that triggers a backtrace with default logging settings.
---
src/responder/common/responder_dp.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/responder/common/responder_dp.c b/src/responder/common/responder_dp.c
index 1b016dba1e..ecf37ab35b 100644
--- a/src/responder/common/responder_dp.c
+++ b/src/responder/common/responder_dp.c
@@ -231,8 +231,9 @@ sss_dp_get_account_send(TALLOC_CTX *mem_ctx,
}
goto done;
} else if (ret != EAGAIN) {
-DEBUG(SSSDBG_OP_FAILURE, "Failed to set files provider update "
- "[%d]: %s\n", ret, sss_strerror(ret));
+DEBUG((ret == ENOENT) ? SSSDBG_MINOR_FAILURE : SSSDBG_OP_FAILURE,
+ "Failed to set files provider update [%d]: %s\n",
+ ret, sss_strerror(ret));
goto done;
}
/* EAGAIN, fall through to issuing the request */
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][-Changes requested] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][comment] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. alexey-tikhonov commented: """ > LGTM, one question: do you think we should mention in the commit message the > reason why file is unused / reason why it is deprecated? I can see that > original commits points back to 2009 and it looks like this piece of code has > been deprecated due to later refactorings and architecture changes. I believe this is a leftover of "sbus" code refactoring. This file wasn't even compiled (or anyhow used) for a few years. Maybe not even included in source tarball. I don't think anyone cares / notices it's removed from upstream repo. """ See the full comment at https://github.com/SSSD/sssd/pull/5934#issuecomment-1004086143 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][comment] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 alexey-tikhonov commented: """ RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1893192 """ See the full comment at https://github.com/SSSD/sssd/pull/5937#issuecomment-1002782096 ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][+Bugzilla] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937 Title: #5937: [WIP] Another attempt to resolve #5134 Label: +Bugzilla ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5937][opened] [WIP] Another attempt to resolve #5134
URL: https://github.com/SSSD/sssd/pull/5937
Author: alexey-tikhonov
Title: #5937: [WIP] Another attempt to resolve #5134
Action: opened
PR body:
"""
There are two ideas:
- get rid of re-alloc for every new member added: instead reserve space with a
margin and shrink later.
- don't add every new member to the list immediately: instead keep new members
in a temporarily hash table. This will allow to avoid cross strcmp() at least
among new members - hopefully, hash table should be much faster.
2nd patch actually makes 1st patch obsolete, but I will keep it a while to
measure impact.
This is "WIP" as I didn't conduct perf measurements yet.
"""
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5937/head:pr5937
git checkout pr5937
From 1557e7f31216919c6f7170d6d503927827e95280 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 28 Dec 2021 23:04:45 +0100
Subject: [PATCH 1/3] SDAP: sdap_nested_group_deref_direct_process(): avoid
realloc for every new member.
Single realloc + shrink are much cheaper.
---
src/providers/ldap/sdap_async_nested_groups.c | 24 ---
1 file changed, 16 insertions(+), 8 deletions(-)
diff --git a/src/providers/ldap/sdap_async_nested_groups.c b/src/providers/ldap/sdap_async_nested_groups.c
index 2570a00bbd..577353569c 100644
--- a/src/providers/ldap/sdap_async_nested_groups.c
+++ b/src/providers/ldap/sdap_async_nested_groups.c
@@ -2334,6 +2334,15 @@ sdap_nested_group_deref_direct_process(struct tevent_req *subreq)
goto done;
}
+/* The same with new members: make an assumpation all entries are new
+ and shrink memory later. */
+members->values = talloc_realloc(members, members->values, struct ldb_val,
+ members->num_values + num_entries);
+if (members->values == NULL) {
+ret = ENOMEM;
+goto done;
+}
+
PROBE(SDAP_NESTED_GROUP_DEREF_PROCESS_PRE);
for (i = 0; i < num_entries; i++) {
ret = sysdb_attrs_get_string(entries[i]->attrs,
@@ -2362,14 +2371,6 @@ sdap_nested_group_deref_direct_process(struct tevent_req *subreq)
/* Append newly found member to member list.
* Changes in state->members will propagate into sysdb_attrs of
* the group. */
-state->members->values = talloc_realloc(members, members->values,
-struct ldb_val,
-members->num_values + 1);
-if (members->values == NULL) {
-ret = ENOMEM;
-goto done;
-}
-
members->values[members->num_values].data =
(uint8_t *)talloc_strdup(members->values, orig_dn);
if (members->values[members->num_values].data == NULL) {
@@ -2452,6 +2453,13 @@ sdap_nested_group_deref_direct_process(struct tevent_req *subreq)
talloc_zfree(state->nested_groups);
}
+members->values = talloc_realloc(members, members->values, struct ldb_val,
+ members->num_values);
+if (members->values == NULL) {
+ret = ENOMEM;
+goto done;
+ }
+
ret = EOK;
done:
From 9cf6a1c868de1b91a2f6f038cbb88cde303f0ce2 Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Wed, 29 Dec 2021 21:04:26 +0100
Subject: [PATCH 2/3] SDAP: sdap_nested_group_deref_direct_process(): don't add
all new members to the list immediately to avoid cross strcasecmp() amongst
them.
Resolves: https://github.com/SSSD/sssd/issues/5134
---
src/providers/ldap/sdap_async_nested_groups.c | 89 ++-
1 file changed, 66 insertions(+), 23 deletions(-)
diff --git a/src/providers/ldap/sdap_async_nested_groups.c b/src/providers/ldap/sdap_async_nested_groups.c
index 577353569c..4c89542b12 100644
--- a/src/providers/ldap/sdap_async_nested_groups.c
+++ b/src/providers/ldap/sdap_async_nested_groups.c
@@ -2293,6 +2293,49 @@ sdap_nested_group_deref_send(TALLOC_CTX *mem_ctx,
return req;
}
+static errno_t
+expand_members_list(struct ldb_message_element *members, hash_table_t *addon)
+{
+struct hash_iter_context_t *iter;
+hash_entry_t *entry;
+const char *dn;
+unsigned long num_new_members = hash_count(addon);
+
+if (num_new_members == 0) {
+return EOK;
+}
+
+members->values = talloc_realloc(members, members->values, struct ldb_val,
+ members->num_values + num_new_members);
+if (members->values == NULL) {
+return ENOMEM;
+}
+
+iter = new_hash_iter_context(addon);
+if (iter == NULL) {
+return ENOMEM;
+}
+
+while ((entry = iter->next(iter)) != NULL) {
+if (entry->key.type != HASH_KEY_CONST_STRING) {
+DEBUG(SSS
[SSSD] [sssd PR#5936][+Waiting for review] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936 Title: #5936: RESPONDER: reduce log level in case inconsistent Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5934][+Waiting for review] Removed unused file.
URL: https://github.com/SSSD/sssd/pull/5934 Title: #5934: Removed unused file. Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5936][opened] RESPONDER: reduce log level in case inconsistent
URL: https://github.com/SSSD/sssd/pull/5936
Author: alexey-tikhonov
Title: #5936: RESPONDER: reduce log level in case inconsistent
Action: opened
PR body:
"""
files provider falls back to NSS
"""
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5936/head:pr5936
git checkout pr5936
From dd69e499cd184fe3cd0e5443720f869e25ea7aec Mon Sep 17 00:00:00 2001
From: Alexey Tikhonov
Date: Tue, 28 Dec 2021 16:30:43 +0100
Subject: [PATCH] RESPONDER: reduce log level in case inconsistent files
provider falls back to NSS
---
src/responder/common/responder_dp.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/responder/common/responder_dp.c b/src/responder/common/responder_dp.c
index 1b016dba1e..ecf37ab35b 100644
--- a/src/responder/common/responder_dp.c
+++ b/src/responder/common/responder_dp.c
@@ -231,8 +231,9 @@ sss_dp_get_account_send(TALLOC_CTX *mem_ctx,
}
goto done;
} else if (ret != EAGAIN) {
-DEBUG(SSSDBG_OP_FAILURE, "Failed to set files provider update "
- "[%d]: %s\n", ret, sss_strerror(ret));
+DEBUG((ret == ENOENT) ? SSSDBG_MINOR_FAILURE : SSSDBG_OP_FAILURE,
+ "Failed to set files provider update [%d]: %s\n",
+ ret, sss_strerror(ret));
goto done;
}
/* EAGAIN, fall through to issuing the request */
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure
[SSSD] [sssd PR#5935][+Bugzilla] sysdb: more specific mpg search filter
URL: https://github.com/SSSD/sssd/pull/5935 Title: #5935: sysdb: more specific mpg search filter Label: +Bugzilla ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
