Tickets :: [#5977] smbldap driver fail in mixed environment

Summary	smbldap driver fail in mixed environment
Queue	Passwd
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester	marco (at) csita (dot) unige (dot) it
Created	12/09/2007 (1993 days ago)
Due
Updated	12/11/2007 (1991 days ago)
Assigned
Resolved	12/11/2007 (1991 days ago)
Attachments	smbldap.php[1].patch
Milestone
Patch	No

12/11/2007 07:26:04 PM	Chuck Hagenbuch	Comment #6 Assigned to Chuck Hagenbuch State ⇒ Resolved	Reply to this comment
Committed, thanks!

12/11/2007 07:20:46 PM	Chuck Hagenbuch	Deleted Attachment: smbldap.php.patch

12/11/2007 02:49:59 PM	marco (at) csita (dot) unige (dot) it	Comment #5 New Attachment: smbldap.php[1].patch	Reply to this comment
I wrongly worked on a previous version, this should be the right one. Cheers, Marco.

12/10/2007 10:25:48 PM	Chuck Hagenbuch	Comment #4	Reply to this comment
Can you make sure the patch is up to date against http://cvs.horde.org/co.php?r=1.16&f=passwd%2Flib%2FDriver%2Fsmbldap.php? The patch doesn't apply cleanly for me and it seems to undo some of the work of ignoring commented out attributes (in the array_merge segment at least). Thanks!

12/10/2007 08:16:00 PM	Chuck Hagenbuch	Deleted Attachment: smbldap.php

12/10/2007 06:54:09 PM	marco (at) csita (dot) unige (dot) it	Comment #3 New Attachment: smbldap.php.patch	Reply to this comment
The entire smbldap.php file is attached, not a patch. Can you please upload a patch? Sorry, correct file is attached herewith. I also realized that every Samba attributes is update separately. I think this is an error, because all attributes must remain in synch and updates should fail or success together. I have correct this behaviour, taking in account also ~~Request #5937~~ http://bugs.horde.org/ticket/?id=5937

12/09/2007 11:07:18 PM	Chuck Hagenbuch	Comment #2 State ⇒ Feedback	Reply to this comment
The entire smbldap.php file is attached, not a patch. Can you please upload a patch?

12/09/2007 01:00:50 PM	marco (at) csita (dot) unige (dot) it	Comment #1 New Attachment: smbldap.php State ⇒ New Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ smbldap driver fail in mixed environment Queue ⇒ Passwd	Reply to this comment
If in LDAP tree entries are mixed among Samba and not Ssamba user, when a non Samba user try to change password using smbldap backend he receives an error ("objectclass violation" usually) but operation success on his standard LDAP password. I attached a patch to apply SMB password update only if entry permit Samba attributes.