On Thu, Jul 30, 2015 at 06:20:31PM +0200, Sumit Bose wrote:
> On Wed, Jul 22, 2015 at 12:18:07PM +0200, Jakub Hrozek wrote:
> > On Wed, Jul 22, 2015 at 10:01:31AM +0200, Sumit Bose wrote:
> > > On Tue, Jul 21, 2015 at 09:41:46PM +0200, Jakub Hrozek wrote:
> > > > Hi,
> > > >
> > > > the attached patch fixes
https://fedorahosted.org/sssd/ticket/2724
for
> > > > me. I haven't yet heard back from the affected user, but they do
have
> > > > test packages available.
> > >
> > > The patch is working as expected and fixes the issue for me as well.
> > >
> > > Although it might not look as the most elegant way I think the approach
> > > is ok because otherwise we would be to check for every group by GID
> > > request if the GID might belong to a MPG which would require a user
> > > lookup. This would slow down the requests where the GID belong to a
> > > normal (non-MPG) group.
> > >
> > > Nevertheless I wonder if we should check if the group name matches the
> > > user name before deleting the group by GID to be on the safe side?
> >
> > OK, done.
>
> Thank you. The patch looks good and is working as advertised, ACK.
sorry, forgot to say that CI is still running, I'll send the result when
it is done.
There is an unrealted issue in rawhide:
RPM build errors:
error: Empty %files file /builddir/build/BUILD/sssd-1.13.1/sssd_client.lang
For details see
> >
> > btw the function is really long. Would you oppose to splitting it into
> > smaller functions that save user/group/... ?
> >
> > (We need tests for that, though..)
>
> no, feel free to open tickets to track this.
>
> bye,
> Sumit
> _______________________________________________
> sssd-devel mailing list
> sssd-devel(a)lists.fedorahosted.org
>
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel