On Wed, Aug 10, 2016 at 08:08:03AM +0200, Petr Cech wrote: > On 08/10/2016 07:30 AM, Petr Cech wrote: > > On 08/09/2016 07:10 PM, Lukas Slebodnik wrote: > > > On (09/08/16 10:21), Jakub Hrozek wrote: > > > > On Mon, Aug 08, 2016 at 06:37:10PM +0200, Lukas Slebodnik wrote: > > > > > ehlo, > > > > > > > > > > yet another patch which fixes issues caused by sysdb refactoring. > > > > > > > > > > reproducer: > > > > > a) add user with two groups > > > > > b) call id user > > > > > c) add another group to user > > > > > d) authenticate > > > > > e) check tha id -G return 3 groups. > > > > > > > > > > I am looking forward to the pam_wrapper So we can test it in upstream. > > > > > > > > > > LS > > > > > > > > > From e501f791ee2e03b8eaea919a2f8ca4474773e3ba Mon Sep 17 00:00:00 2001 > > > > > From: Lukas Slebodnik <lsleb...@redhat.com> > > > > > Date: Mon, 8 Aug 2016 17:30:29 +0200 > > > > > Subject: [PATCH] NSS: Use correct name for invalidating memory cache > > > > > > > > > > After refactoring of sysdb, we get and internal fully qualified > > > > > name from backend in org.freedesktop.sssd.dataprovider_rev.initgrCheck > > > > > Previously we got short name and we created fq name in > > > > > nss_update_initgr_memcache. Memory cache still need to use short names > > > > > if it was specified. > > > > > > > > In fill_pwent() we use the sized_output_name() to store the name in the > > > > memcache, can we use the same here? > > > Sure, > > > > > > LS > > > > Hi Lukas, Jakub, > > > > LGTM, I will wait for new CI. > > Hi, > > CI passed: > http://sssd-ci.duckdns.org/logs/job/51/22/summary.html > > Manual testing passed. > > => ACK
ACK from me as well. * master: 6f0a0ae7599e3947c0b2e4649039f85829e57637 _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org