Tickets :: [#12299] Simple fix for ldap->findUserDN

6.0.0-alpha10

5/14/25

Summary	Simple fix for ldap->findUserDN
Queue	Passwd
Queue Version	Git master
Type	Enhancement
State	Feedback
Priority	1. Low
Owners
Requester	fzipi (at) fing (dot) edu (dot) uy
Created	06/04/2013 (4362 days ago)
Due
Updated	08/11/2016 (3198 days ago)
Assigned
Resolved
Milestone
Patch	No

08/11/2016 10:06:27 AM	Jan Schneider	Comment #7	Reply to this comment
~~Ticket #11697~~ has been resolved, so we can progress here as suggested.

01/28/2016 04:06:17 PM	Jan Schneider	Comment #6	Reply to this comment
See ~~ticket #11697~~ too.

01/28/2016 02:53:20 PM	Jan Schneider	Comment #5	Reply to this comment
I wonder if we shouldn't always use findUserDN() and drop the 'userdn' setting and hook altogether?

11/20/2013 02:29:13 PM	fzipi (at) fing (dot) edu (dot) uy	Comment #4 New Attachment: 0001-Adding-config-parameter-to-check-if-user-should-be-s.patch	Reply to this comment
With last changes now the 'user' parameter is correctly propagated to the LDAP driver. As there is no need to hardcode this change, I've added a new config parameter 'search_userdn' in backends.php. If this setting is true (in your 'backends.local.php'), it will search the user's DN using 'findUserDN'. Otherwise, it will behave as usual.

06/11/2013 11:34:37 AM	Jan Schneider	Comment #3 State ⇒ Feedback Patch ⇒ No	Reply to this comment
Hardcoding default values is not a fix. The question is, why is the 'user' parameter not correctly propagated to the LDAP driver.

06/04/2013 10:05:20 PM	fzipi (at) fing (dot) edu (dot) uy	Comment #2 New Attachment: 0002-Added-user-parameter-so-findUserDN-could-be-used-alo.patch	Reply to this comment
I've forgot to add some sane defaults.

06/04/2013 10:04:03 PM	fzipi (at) fing (dot) edu (dot) uy	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Simple fix for ldap->findUserDN Queue ⇒ Passwd Milestone ⇒ Patch ⇒ Yes State ⇒ New	Reply to this comment
We are using ldap for our users, but in many different ou's from base. Therefore we can't use userdn config param. I've managed to get the ldap password changing to work using this simple patch. Horde_Ldap uses $params['user'] with an array to do the actual search. When defined, use values from config file 'uid' and 'filter' for construct search parameters 'user'.