[#12299] Simple fix for ldap->findUserDN
Summary Simple fix for ldap->findUserDN
Queue Passwd
Queue Version Git master
Type Enhancement
State Feedback
Priority 1. Low
Owners
Requester fzipi@fing.edu.uy
Created 2013-06-04 (2240 days ago)
Due
Updated 2016-08-11 (1076 days ago)
Assigned
Resolved
Milestone
Patch No

Comments
fzipi@fing.edu.uy 2013-06-04 22:04:03
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'.



fzipi@fing.edu.uy 2013-06-04 22:05:20
I've forgot to add some sane defaults.


Jan Schneider <jan@horde.org> 2013-06-11 11:34:37
Hardcoding default values is not a fix. The question is, why is the 
'user' parameter not correctly propagated to the LDAP driver.

fzipi@fing.edu.uy 2013-11-20 14:29:13
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.

Jan Schneider <jan@horde.org> 2016-01-28 14:53:20
I wonder if we shouldn't always use findUserDN() and drop the 'userdn' 
setting and hook altogether?

Jan Schneider <jan@horde.org> 2016-01-28 16:06:17
See ticket #11697 too.

Jan Schneider <jan@horde.org> 2016-08-11 10:06:27
Ticket #11697 has been resolved, so we can progress here as suggested.