[SSSD] [PATCHES] Merging INI snippets

2014-09-10 Thread Dmitri Pal
Hello, Extensive travel in recent months allowed me to finish this code. Here is the updated design: https://fedorahosted.org/sssd/wiki/DesignDocs/ding-libs/INIConfigMerge Patches: 0001 - Fix in case the Ref array is empty and we need to print/debug it 0002 - Declaration of the new function to

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Michal Židek
On 09/10/2014 06:13 PM, Michal Židek wrote: On 09/10/2014 06:00 PM, Michal Židek wrote: On 09/10/2014 02:49 PM, Jakub Hrozek wrote: On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -348,9 +348,12 @@ int sdap_parse_

Re: [SSSD] [PATCH] AD: conflicting gpo policy settings not being resolved correctly

2014-09-10 Thread Yassir Elley
- Original Message - > Hi, > > The attached patch fixes ticket #2437 (conflicting gpo policy settings not > being resolved correctly) > > https://fedorahosted.org/sssd/ticket/2437 > > Regards, > Yassir. > > ___ > sssd-devel mailing list > ss

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Michal Židek
On 09/10/2014 06:00 PM, Michal Židek wrote: On 09/10/2014 02:49 PM, Jakub Hrozek wrote: On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -348,9 +348,12 @@ int sdap_parse_entry(TALLOC_CTX *memctx, }

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Michal Židek
On 09/10/2014 02:49 PM, Jakub Hrozek wrote: On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: --- a/src/providers/ldap/sdap.c +++ b/src/providers/ldap/sdap.c @@ -348,9 +348,12 @@ int sdap_parse_entry(TALLOC_CTX *memctx, } for (i = 0; vals[i]; i++) { -

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 04:14:40PM +0200, Jakub Hrozek wrote: > But I think we should have a proper discussion on this list. Sorry, I didn't mean we're not having one now :-) What I meant is we should propose what we think is the best way in a short design page, maybe along with the future filter

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 04:00:29PM +0200, Sumit Bose wrote: > On Wed, Sep 10, 2014 at 03:06:07PM +0200, Jakub Hrozek wrote: > > On Wed, Sep 10, 2014 at 02:55:34PM +0200, Sumit Bose wrote: > > > On Wed, Sep 10, 2014 at 02:33:00PM +0200, Jakub Hrozek wrote: > > > > On Wed, Sep 10, 2014 at 02:22:15PM

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Sumit Bose
On Wed, Sep 10, 2014 at 03:06:07PM +0200, Jakub Hrozek wrote: > On Wed, Sep 10, 2014 at 02:55:34PM +0200, Sumit Bose wrote: > > On Wed, Sep 10, 2014 at 02:33:00PM +0200, Jakub Hrozek wrote: > > > On Wed, Sep 10, 2014 at 02:22:15PM +0200, Sumit Bose wrote: > > > > On Wed, Sep 10, 2014 at 01:23:15PM

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 02:55:34PM +0200, Sumit Bose wrote: > On Wed, Sep 10, 2014 at 02:33:00PM +0200, Jakub Hrozek wrote: > > On Wed, Sep 10, 2014 at 02:22:15PM +0200, Sumit Bose wrote: > > > On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: > > > > Hi, > > > > > > > > this patch sho

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Sumit Bose
On Wed, Sep 10, 2014 at 02:33:00PM +0200, Jakub Hrozek wrote: > On Wed, Sep 10, 2014 at 02:22:15PM +0200, Sumit Bose wrote: > > On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: > > > Hi, > > > > > > this patch should solve regression on IPA server > > > triggered by the patches for th

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: > --- a/src/providers/ldap/sdap.c > +++ b/src/providers/ldap/sdap.c > @@ -348,9 +348,12 @@ int sdap_parse_entry(TALLOC_CTX *memctx, > } > > for (i = 0; vals[i]; i++) { > -/* the objectclass is always the f

Re: [SSSD] [PATCH] LDAP: Don't reuse a single tevent callback for multiple requests

2014-09-10 Thread Michal Židek
On 09/10/2014 12:17 PM, Jakub Hrozek wrote: On Wed, Sep 10, 2014 at 11:59:55AM +0200, Jakub Hrozek wrote: On Fri, Sep 05, 2014 at 04:28:00PM +0200, Jakub Hrozek wrote: I will prepare a 1.11 version separately (sadly, the bug is also there) Luckily it is not :-) but the referrals are not igno

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 02:22:15PM +0200, Sumit Bose wrote: > On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: > > Hi, > > > > this patch should solve regression on IPA server > > triggered by the patches for this ticket: > > https://fedorahosted.org/sssd/ticket/2343 > > > > Patches

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Sumit Bose
On Wed, Sep 10, 2014 at 01:23:15PM +0200, Michal Židek wrote: > Hi, > > this patch should solve regression on IPA server > triggered by the patches for this ticket: > https://fedorahosted.org/sssd/ticket/2343 > > Patches for #2343 solved the IPA side of the > problem by searching for objectclass

Re: [SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Pavel Reichl
Hello Michal, code LGTM but I haven't tested it yet. Could you add ticket number (https://fedorahosted.org/sssd/ticket/2436) to the patch? Thanks, Pavel On 09/10/2014 01:23 PM, Michal Židek wrote: Hi, this patch should solve regression on IPA server triggered by the patches for this tick

[SSSD] [PATCHES] Alternative objectClass for group maps

2014-09-10 Thread Michal Židek
Hi, this patch should solve regression on IPA server triggered by the patches for this ticket: https://fedorahosted.org/sssd/ticket/2343 Patches for #2343 solved the IPA side of the problem by searching for objectclass groupOfNames instead of posixGroup (which was not available in non-posix grou

[SSSD] [PATCH] AD: conflicting gpo policy settings not being resolved correctly

2014-09-10 Thread Yassir Elley
Hi, The attached patch fixes ticket #2437 (conflicting gpo policy settings not being resolved correctly) https://fedorahosted.org/sssd/ticket/2437 Regards, Yassir. From a3e0abad5341b96aa1909082778bd1abb6746c5a Mon Sep 17 00:00:00 2001 From: Yassir Elley Date: Tue, 9 Sep 2014 15:37:05 -0400 Sub

Re: [SSSD] [PATCH] LDAP: Don't reuse a single tevent callback for multiple requests

2014-09-10 Thread Jakub Hrozek
On Wed, Sep 10, 2014 at 11:59:55AM +0200, Jakub Hrozek wrote: > On Fri, Sep 05, 2014 at 04:28:00PM +0200, Jakub Hrozek wrote: > > I will prepare a 1.11 version separately (sadly, the bug is also there) > > Luckily it is not :-) > > but the referrals are not ignored in sssd-1-11 for deref and ASQ,

Re: [SSSD] [PATCH] LDAP: Don't reuse a single tevent callback for multiple requests

2014-09-10 Thread Jakub Hrozek
On Fri, Sep 05, 2014 at 04:28:00PM +0200, Jakub Hrozek wrote: > I will prepare a 1.11 version separately (sadly, the bug is also there) Luckily it is not :-) but the referrals are not ignored in sssd-1-11 for deref and ASQ, which is what this patch implements. >From c7ed79f27fcc36b2a7a0f47222e1f0