[SSSD] [PATCHES] Add SID related lookups to the NSS responder - part 2

Jakub Hrozek jhrozek at redhat.com
Fri May 3 17:59:07 UTC 2013


On Fri, May 03, 2013 at 06:39:17PM +0200, Sumit Bose wrote:
> On Fri, May 03, 2013 at 01:31:12PM +0200, Sumit Bose wrote:
> > On Fri, May 03, 2013 at 12:27:32PM +0200, Jakub Hrozek wrote:
> > > On Fri, May 03, 2013 at 09:27:29AM +0200, Sumit Bose wrote:
> > > > On Thu, May 02, 2013 at 08:31:40PM +0200, Sumit Bose wrote:
> > > > > On Thu, May 02, 2013 at 07:23:11PM +0200, Jakub Hrozek wrote:
> > > > > > On Thu, May 02, 2013 at 04:07:57PM +0200, Sumit Bose wrote:
> > > > > > > Hi,
> > > > > > > 
> > > > > > > this is the second series of patches for the SID related lookups. With
> > > > > > > these 4 patches, together with the ones send before, the FreeIPA WebUI
> > > > > > > can do the SID-to-name lookups as described in
> > > > > > > https://fedorahosted.org/freeipa/ticket/3302.
> > > > > > > 
> > > > > > > The patches currently only support IPA subdomain user, i.e. user and
> > > > > > > groups from trusted domains. Upcoming patches will add support for IPA
> > > > > > > user and support for the AD provider as well.
> > > > > > > 
> > > > > > > bye,
> > > > > > > Sumit
> > > > > > 
> > > > > > Seems like the patches need rebasing on top of Pavel's recent patchset..
> > > > > 
> > > > > rebased on current master.
> > > > > 
> > > 
> > > Patch 0003 still doesn't apply on master for me.. (sorry, I think it's
> > > because I pushed Abhishek's tests after you rebased)
> > 
> > I realized that there also was a patch in my tree which might have
> > caused the issue. New version attached.
> > 
> > bye,
> > Sumit
> 
> Jakub send me some review comments off-list which I added to the second
> patch. New versions attached.
> 
> I added a 5th patch which adds support for tuples and Unicode to the
> python API as inquired by Alexander. I didn't squeezed them because I'm
> not sure if Alexander was already able to test them. This way they can
> easily be dropped from this release and can be added to the next.
> 
> bye,
> Sumit

Patches 1-4: Ack

I haven't reviewed patch #5 yet.



More information about the sssd-devel mailing list