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
to comment on this ticket.
Copyright © 2014-2017 Red Hat
2.90.1 — Documentation