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.