Hi,
the attached patch implements Sumit's idea to solve https://fedorahosted.org/sssd/ticket/2927
The user who reported the bug confirmed that the patch works. As an additional improvement, they requested that these results should not be considered when the cache expires, but immediatelly. But honestly I don't know if and how this could be solved (we would need to fetch this attribute always on initgroups on both client and server..) so I would prefer additional ticket and merge this patch first.
On Wed, May 11, 2016 at 11:58:11AM +0200, Jakub Hrozek wrote:
Hi,
the attached patch implements Sumit's idea to solve https://fedorahosted.org/sssd/ticket/2927
The user who reported the bug confirmed that the patch works. As an additional improvement, they requested that these results should not be considered when the cache expires, but immediatelly. But honestly I don't know if and how this could be solved (we would need to fetch this attribute always on initgroups on both client and server..) so I would prefer additional ticket and merge this patch first.
ping; a downstream is waiting for this patch..
On 05/17/2016 03:11 PM, Jakub Hrozek wrote:
On Wed, May 11, 2016 at 11:58:11AM +0200, Jakub Hrozek wrote:
Hi,
the attached patch implements Sumit's idea to solve https://fedorahosted.org/sssd/ticket/2927
The user who reported the bug confirmed that the patch works. As an additional improvement, they requested that these results should not be considered when the cache expires, but immediatelly. But honestly I don't know if and how this could be solved (we would need to fetch this attribute always on initgroups on both client and server..) so I would prefer additional ticket and merge this patch first.
Agreed.
ping; a downstream is waiting for this patch..
Ack.
On Wed, May 18, 2016 at 10:29:37AM +0200, Pavel Březina wrote:
On 05/17/2016 03:11 PM, Jakub Hrozek wrote:
On Wed, May 11, 2016 at 11:58:11AM +0200, Jakub Hrozek wrote:
Hi,
the attached patch implements Sumit's idea to solve https://fedorahosted.org/sssd/ticket/2927
The user who reported the bug confirmed that the patch works. As an additional improvement, they requested that these results should not be considered when the cache expires, but immediatelly. But honestly I don't know if and how this could be solved (we would need to fetch this attribute always on initgroups on both client and server..) so I would prefer additional ticket and merge this patch first.
Agreed.
ping; a downstream is waiting for this patch..
Ack.
Thank you for the review and sorry I didn't push the patch sooner, I completely missed the ack.
* master: eedfc2cced329731c90317a5be3cd82a3749eb8a * sssd-1-13: fdb1b73dfeb25675a7f4f11172bee9a784ccc054
Could someone please close the ticket for me? My fedora account has been locked out :-)
sssd-devel@lists.fedorahosted.org
-
Jakub Hrozek -
Pavel Březina