On Tue, Apr 22, 2014 at 09:23:41AM +0200, Pavel Březina wrote:
On 04/21/2014 11:32 PM, Jakub Hrozek wrote:
>Hi,
>
>the NSS and PAM responders copy quite a bit of code around for the
>purpose of splitting an input string into name-domain components with
>optionally firing a request to refresh subdomains.
>
>I think the caller should not do all these chores, but instead just call
>an opaque request that will decide whether to refresh subdomains or
>whether to reuse the cached list. Instead of adding even more
>duplication to the DBus responder, I've implemented such request, did a
>minor cleanup (removing of the DP force flag) and added a bunch of unit
>tests.
>
>If these patches are accepted, I would like to convert the NSS and PAM
>responder code for parsing input to this new request.
Ack to all. Nice work.
Thanks for the review, pushed to master: 03a0713..7caf7ed
I also filed a ticket to use this request in other responders so that we
don't forget:
https://fedorahosted.org/sssd/ticket/2320