On Wed, May 21, 2014 at 03:15:04PM +0200, Jakub Hrozek wrote:
> On Wed, May 21, 2014 at 03:58:52PM +0300, Alexander Bokovoy wrote:
> > On Tue, 20 May 2014, Jakub Hrozek wrote:
> > >On Tue, May 13, 2014 at 11:45:29AM +0200, Pavel Reichl wrote:
> > >>Hello,
> > >>
> > >>Alexander has prepared attached patches, but he was not able to post
> > >>them himself as he is currently having bad connection.
> > >>
> > >>Pavel Reichl
> > >
> > >I've tested the patches atop my patch called "AD: Initialize
> > >user_map_cnt in server mode" that I sent to master today. It's a good
> > >start but I would like to propose two amendments that I attach.
> > >
> > >One makes sure the search by SID also includes UID which we read later,
> > >the second makes sure ENOENT or an empty message is handled gracefully
> > >and doesn't pollute the logs with a DEBUG message.
> > >
> > >If you agree, we can apply these patches atop yours or squash them in, I
> > >don't mind either way.
> > >
> > >The second patch looks good to me.
> > ACK to both. I'm fine with squashing the patches together.
>
> Thanks for the review, for posterity I have attached the patches I'm
> going to push.
Pushed to master:
58092eb1acfefefef9148789062aa9114e00372e
33d70148eac28c548b10eb3aa36ab23169cedc5a
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
