Re: [vchkpw] additional ldap changes (mail attribute) to patch 1609348

2008-01-09 Thread Rick Widmer

I'll take a look at this over the weekend...  Sorry I'm so slow right now.

Ismail YENIGUL wrote:


!DSPAM:4785a19f310542282815610!



[vchkpw] additional ldap changes (mail attribute) to patch 1609348

2008-01-08 Thread Ismail YENIGUL
Hello Again,


Previously, a patch was submitted by Vitali Malicky about  LDAP changes with 
the patch id: 1609348 

url:https://sourceforge.net/tracker/?func=detail&atid=577800&aid=1609348&group_id=85937

According to the changelog of vpopmail, This patch was added to vpopmail 5.4.20.
When I checked vpopmail 5.4.25, I recognized that this patch is not really 
added.

I made some changes on this patch to be compatible with Addressbook stuffs.

1. Vitali was assigning [EMAIL PROTECTED] as cn value, I changed it with GECOS 
value. (because, addressbooks consider cn as a full name of the person)

2. I added new ldap attribute (mail) to be able to search mail
addresses over the addressbook.


As a result, with this patch, vpopmail will be compatible with OpenLDAP 2.3.X 
and have a basic addressbook feature.

You can find the patch at 
https://sourceforge.net/tracker/index.php?func=detail&aid=1867246&group_id=85937&atid=577800

Thanks.


Wednesday, January 2, 2008, 11:43:28 AM, you wrote:

> Hello,

> According to the ChangeLog, the patch name in the subject was added to
> vpopmail 5.4.20

> 5.4.20 - Released 21-Aug-2007
>  Vitali Malicky (coonardoo), Ismail Yenigul
> - add ldap patch [1909348]

> But, when I tried to test vpopmail-5.4.20 and 5.4.26 with ldap support 
> (openldap 2.3.X),
> I got the "qmailUser' requires attribute 'sn'" error. I recognized
> that the patch is not added into vpopmail.

> /* Background info: This patch introduces sn and cn ldap attributes to be 
> able to use
> openldap 2.3.X.  and sn and cn attributes MUST in qmailUser.schema) */

> Normally, If the patch was added to vpopmail, we should have the
> following lines in the vldap.[ch].
> +   "cn",   /* 7 LDAP's "cn" demanded
> by "objectClass: qmailUser"   [EMAIL PROTECTED] */
> +   "sn",   /* 8 LDAP's "cn" demanded
> by "objectClass: qmailUser"   [EMAIL PROTECTED] */

+lm[7]->>mod_values[0] = safe_strdup(inpw->pw_cn);   /* "cn" [EMAIL 
PROTECTED] */
+lm[8]->>mod_values[0] = safe_strdup(inpw->pw_sn);   /* "sn" [EMAIL 
PROTECTED] */

> etc.

> patch url:
> https://sourceforge.net/tracker/?func=detail&atid=577800&aid=1609348&group_id=85937




-- 

Ismail YENIGUL
Yönetici/Manager
Servis ve Destek Bölümü/System Services and Support Division
Tel: +90 216 344 07 15
Fax: +90 216 344 07 16
http://www.endersys.com.tr


!DSPAM:4783f7b4310541964932056!