#3328 Use search_all_domains in the list IFP methods
Closed: cloned-to-github 3 years ago by pbrezina. Opened 7 years ago by jhrozek.

This came up during code review of PR #174, but we decided to not off merging of the code because it's a cosmetic code issue. The code currently iterates over all domains, gathering the requests. Pavel suggested the following:

The intention here is to gather result from all domains, right? I implemented search_all_domains plugin option for enumeration use case to allow cache_req to not stop on the first domain with result but get result from all domains.

This can be used in ifp_users_list_by_name instead of iteration, we just need to switch the filter plugins to this.

We can also use it for the method implemented here. cache_req needs small work though to allow override plugin settings in similar way Fabiano did for PAM responder (cache_req_data_set_bypass_cache).


Metadata Update from @jhrozek:
- Issue set to the milestone: SSSD 1.15.4

6 years ago

Metadata Update from @jhrozek:
- Issue tagged with: easyfix

6 years ago

Metadata Update from @jhrozek:
- Issue tagged with: cleanup-future, refactoring

6 years ago

Metadata Update from @jhrozek:
- Issue untagged with: cleanup-future
- Issue set to the milestone: SSSD Future releases (no date set yet) (was: SSSD 1.15.4)

6 years ago

Metadata Update from @thalman:
- Issue tagged with: Future milestone

4 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/4359

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @pbrezina:
- Issue close_status updated to: cloned-to-github
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata