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