#3328 Use search_all_domains in the list IFP methods

Created 8 months ago by jhrozek
Modified 3 months ago

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).

8 months ago

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

4 months ago

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

3 months ago

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

3 months 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)

Login to comment on this ticket.

cancel