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
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to