OpenLDAP
Up to top level
Build   Contrib   Development   Documentation   Historical   Incoming   Software Bugs   Software Enhancements   Web  

Logged in as guest

Viewing Software Enhancements/6536
Full headers

From: raphael.ouazana@linagora.com
Subject: Autogroup overlay enhancement
Compose comment
Download message
State:
0 replies:
2 followups: 1 2

Major security issue: yes  no

Notes:

Notification:


Date: Tue, 27 Apr 2010 10:39:29 +0000
From: raphael.ouazana@linagora.com
To: openldap-its@OpenLDAP.org
Subject: Autogroup overlay enhancement
Full_Name: Raphael Ouazana
Version: 2.4.21
OS: Linux
URL: ftp://ftp.openldap.org/incoming/raphael-ouazana-autogroup-100427.patch
Submission from: (NULL) (213.41.232.151)


Hi,

The attached patch allow the autogroup overlay to handle correctly the attr part
of the URL ldap://dc=example,dc=com?attr?sub?(filter). Before this patch, the
attr part was simply ignored. Now the group entry is populated by the values of
the attribute attr in the resulting entries.

Implementation details:
Some cases (modify, delete) are harder to handle when you store values instead
of dn. In this cases the overlay try to detect if groups have been modified and
then simply refresh them. This can cause performance hits if the search
specified by the URL deals with an important number of entries.

Legal notice:
This patch file is derived from OpenLDAP Software. All of the modifications to
OpenLDAP Software represented in this following patch were developed by Raphael
Ouazana raphael.ouazana@linagora.com. These modifications are not subject to
any
license of Linagora.

The attached modifications to OpenLDAP Software are subject to the following
notice:
Copyright 2010 Raphael Ouazana, Linagora
Redistribution and use in source and binary forms, with or without
modification,
are permitted only as authorized by the OpenLDAP Public License. 

Followup 1

Download message
Date: Tue, 01 Jun 2010 18:44:49 -0700
From: Howard Chu <hyc@symas.com>
To: raphael.ouazana@linagora.com
CC: openldap-its@openldap.org
Subject: Re: (ITS#6536) Autogroup overlay enhancement
raphael.ouazana@linagora.com wrote:
> Full_Name: Raphael Ouazana
> Version: 2.4.21
> OS: Linux
> URL: ftp://ftp.openldap.org/incoming/raphael-ouazana-autogroup-100427.patch
> Submission from: (NULL) (213.41.232.151)
>
>
> Hi,
>
> The attached patch allow the autogroup overlay to handle correctly the attr
part
> of the URL ldap://dc=example,dc=com?attr?sub?(filter). Before this patch,
the
> attr part was simply ignored. Now the group entry is populated by the
values of
> the attribute attr in the resulting entries.
>
> Implementation details:
> Some cases (modify, delete) are harder to handle when you store values
instead
> of dn. In this cases the overlay try to detect if groups have been modified
and
> then simply refresh them. This can cause performance hits if the search
> specified by the URL deals with an important number of entries.
>
> Legal notice:
> This patch file is derived from OpenLDAP Software. All of the modifications
to
> OpenLDAP Software represented in this following patch were developed by
Raphael
> Ouazana raphael.ouazana@linagora.com. These modifications are not subject
to
> any
> license of Linagora.
>
> The attached modifications to OpenLDAP Software are subject to the
following
> notice:
> Copyright 2010 Raphael Ouazana, Linagora
> Redistribution and use in source and binary forms, with or without
> modification,
> are permitted only as authorized by the OpenLDAP Public License.

The code patch looks OK to me, but could you also provide a patch for the 
README file summarizing this new behavior? I'm not too keen on supporting 
groups of anything other than DNs in general, but I see the usefulness of it. 
However the potential performance hit can be quite a concern.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/



Followup 2

Download message
Date: Wed, 2 Jun 2010 11:31:18 +0200
Subject: Re: (ITS#6536) Autogroup overlay enhancement
From: =?iso-8859-1?Q?Rapha=EBl_Ouazana-Sustowski?= <raphael.ouazana@linagora.com>
To: "Howard Chu" <hyc@symas.com>
Cc: openldap-its@openldap.org
Le Mer 2 juin 2010 03:44, Howard Chu a .crit :
> The code patch looks OK to me, but could you also provide a patch for the
> README file summarizing this new behavior? I'm not too keen on supporting
> groups of anything other than DNs in general, but I see the usefulness of
> it.
> However the potential performance hit can be quite a concern.

Thank you for your feedback. Here is the patch of the README file:
ftp://ftp.openldap.org/incoming/raphael-ouazana-autogroup-readme-1000602.patch

Regards,
Rapha.l Ouazana.


Up to top level
Build   Contrib   Development   Documentation   Historical   Incoming   Software Bugs   Software Enhancements   Web  

Logged in as guest


The OpenLDAP Issue Tracking System uses a hacked version of JitterBug

______________
© Copyright 2013, OpenLDAP Foundation, info@OpenLDAP.org