Tickets :: [#7839] auth ldap driver does not use a scope search preference

6.0.0-beta1

7/3/25

Summary	auth ldap driver does not use a scope search preference
Queue	Horde Framework Packages
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	dom.lalot (at) gmail (dot) com
Created	01/12/2009 (6016 days ago)
Due
Updated	04/04/2009 (5934 days ago)
Assigned	01/27/2009 (6001 days ago)
Resolved	04/04/2009 (5934 days ago)
Milestone	3.3.4
Patch	Yes

04/04/2009 10:07:15 AM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Auth/Auth/ldap.php?rt=horde&r1=1.47.10.29&r2=1.47.10.30&ty=u http://cvs.horde.org/diff.php/horde/config/conf.xml?rt=horde&r1=1.74.2.81&r2=1.74.2.82&ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.515.2.519&r2=1.515.2.520&ty=u

04/04/2009 09:39:15 AM	Jan Schneider	Comment #6 Assigned to Jan Schneider Taken from Horde Developers State ⇒ Resolved	Reply to this comment
Committed, thanks.

04/04/2009 09:39:10 AM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Auth/Auth/ldap.php?rt=horde&r1=1.92&r2=1.93&ty=u http://cvs.horde.org/diff.php/horde/config/conf.xml?rt=horde&r1=1.248&r2=1.249&ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.1219&r2=1.1220&ty=u

01/27/2009 02:47:58 PM	Jan Schneider	Patch ⇒ Yes Milestone ⇒ 3.3.4 Priority ⇒ 2. Medium State ⇒ Assigned Assigned to Horde Developers

01/27/2009 02:47:32 PM	Jan Schneider	Queue ⇒ Horde Framework Packages Version ⇒ HEAD

01/22/2009 03:59:50 PM	dom (dot) lalot (at) gmail (dot) com	Comment #4 New Attachment: Nouveau Document texte.txt	Reply to this comment
Could you consolidate those suggestions into a patch? I've got no feedback, but I joined a little patch Here it is. Just conf.xml and ldap Auth Dom

01/12/2009 04:24:20 PM	dom (dot) lalot (at) gmail (dot) com	Comment #3	Reply to this comment
Could you consolidate those suggestions into a patch? Well, Why not, but I have to modify lots of files. Just try a grep on ldap_search or ldap_list, you will see a big difference. I hate php ldap api, in perl, scope is just a parameter. Adding that functionality is not easy. I've seen there is a pear object choosing ldap function at object creation. I'm waiting for your suggestions Dom

01/12/2009 04:02:19 PM	Chuck Hagenbuch	Comment #2 State ⇒ Feedback	Reply to this comment
Could you consolidate those suggestions into a patch?

01/12/2009 01:16:34 PM	dom (dot) lalot (at) gmail (dot) com	Comment #1 Priority ⇒ 1. Low State ⇒ New Patch ⇒ No Milestone ⇒ Queue ⇒ Horde Groupware Webmail Edition Summary ⇒ auth ldap driver does not use a scope search preference Type ⇒ Enhancement	Reply to this comment
Hello, There's no possibilities to furnish a scope for an ldap search (horde/config/conf.xml has no value for that), and trying to force a value does not work either in composite Using composite: $conf['auth']['params']['drivers']['ldap_drv']['params']['scope'] ='one'; listUsers() is returning entries of a subtree search lib/Horde/Auth/ldap.php function listUsers() { /* Connect to the LDAP server. */ $result = $this->_connect(); if (is_a($result, 'PEAR_Error')) { return $result; } $filter = $this->_getParamFilter(); There should be a test here (search or list) $search = @ldap_search($this->_ds, $this->_params['basedn'], $filter, array($this->_params['uid'])); Old accounts are stored in ou=old,ou=people and scope should be one for us. It also speeds the search