6.0.0-git
2018-12-16

[#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 (at) fing (dot) edu (dot) uy
Created 2013-06-04 (2021 days ago)
Due
Updated 2016-08-11 (857 days ago)
Assigned
Resolved
Milestone
Patch No

History
2016-08-11 10:06:27 Jan Schneider Comment #7 Reply to this comment
Ticket #11697 has been resolved, so we can progress here as suggested.
2016-01-28 16:06:17 Jan Schneider Comment #6 Reply to this comment
See ticket #11697 too.
2016-01-28 14:53:20 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?
2013-11-20 14:29:13 fzipi (at) fing (dot) edu (dot) uy Comment #4
New Attachment: 0001-Adding-config-parameter-to-check-if-user-should-be-s.patch Download
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.
2013-06-11 11:34:37 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.
2013-06-04 22:05:20 fzipi (at) fing (dot) edu (dot) uy Comment #2
New Attachment: 0002-Added-user-parameter-so-findUserDN-could-be-used-alo.patch Download
Reply to this comment
I've forgot to add some sane defaults.

2013-06-04 22:04:03 fzipi (at) fing (dot) edu (dot) uy Comment #1
Type ⇒ Enhancement
State ⇒ New
Priority ⇒ 1. Low
Summary ⇒ Simple fix for ldap->findUserDN
Queue ⇒ Passwd
Milestone ⇒
Patch ⇒ Yes
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'.


Saved Queries