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

6.0.0-beta6

4/10/26

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	1/12/09 (6297 days ago)
Due
Updated	4/4/09 (6215 days ago)
Assigned	1/27/09 (6282 days ago)
Resolved	4/4/09 (6215 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