6.0.0-beta1
7/27/25

[#3007] Default for LDAP version should be 3
Summary Default for LDAP version should be 3
Queue Horde Base
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners
Requester kevin_myer (at) iu13 (dot) org
Created 11/19/2005 (7190 days ago)
Due
Updated 11/21/2005 (7188 days ago)
Assigned
Resolved 11/21/2005 (7188 days ago)
Milestone
Patch No

History
11/21/2005 09:49:47 PM Jan Schneider Comment #8
State ⇒ Resolved
Reply to this comment
Committed.
11/21/2005 09:45:43 PM kevin_myer (at) iu13 (dot) org Comment #7 Reply to this comment
Ah, I thought all configurations were defaulting to 2. I reckon that
with your patch *all* LDAP configurations default to 3 then, right?
That's correct.
11/21/2005 09:36:13 PM Jan Schneider Comment #6 Reply to this comment
Auth, Prefs, and Groups, and the Account Block all have code to set
the version to $conf['foo']['params']['version'].  Ingo and Turba
both have config files (backends.php, sources.php) that default to 3
and the driver code checks to see what version is specified in the
respective config file.  So I'm not sure what else you need for this.
Ah, I thought all configurations were defaulting to 2. I reckon that 
with your patch *all* LDAP configurations default to 3 then, right?
11/21/2005 08:59:19 PM kevin_myer (at) iu13 (dot) org Comment #5 Reply to this comment
We're still using OpenLDAP 1.2.x (with LDAPv2) for our address book
LDAP.  Though we're soon moving to OpenLDAP 2.x (with LDAPv3 as the
default), I imagine there are others in our situation.
Well that answers the version deprecation question then.. :)
11/21/2005 08:38:57 PM Matt Selsky Comment #4 Reply to this comment
We're still using OpenLDAP 1.2.x (with LDAPv2) for our address book 
LDAP.  Though we're soon moving to OpenLDAP 2.x (with LDAPv3 as the 
default), I imagine there are others in our situation.
11/21/2005 08:25:10 PM kevin_myer (at) iu13 (dot) org Comment #3 Reply to this comment
No idea, but there are probably a lot more places in Horde and it's
apps where we set the protocol version. Please upload a patch for all
of them (in one file).
I was just looking to clean up the conf.xml file, which has two LDAP 
config items defaulting to version 3 and two LDAP config items 
defaulting to version 2, with a goal of making the defaults be 
internally consistent.



Auth, Prefs, and Groups, and the Account Block all have code to set 
the version to $conf['foo']['params']['version'].  Ingo and Turba both 
have config files (backends.php, sources.php) that default to 3 and 
the driver code checks to see what version is specified in the 
respective config file.  So I'm not sure what else you need for this.



And then my question is if no one is using LDAP version 2 anymore, 
should it go beyond just stating that its deprectated but actually 
remove it as an option?  Although there's probably someone out there 
chugging along with an ancient directory server that it would break 
things for...
11/20/2005 12:03:30 AM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
No idea, but there are probably a lot more places in Horde and it's 
apps where we set the protocol version. Please upload a patch for all 
of them (in one file).
11/19/2005 10:55:35 PM kevin_myer (at) iu13 (dot) org Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Default for LDAP version should be 3
Queue ⇒ Horde Base
State ⇒ New
Reply to this comment
(Does anyone even run LDAP v2 servers anymore??)



Index: conf.xml

===================================================================

RCS file: /repository/horde/config/conf.xml,v

retrieving revision 1.120

diff -u -r1.120 conf.xml

--- conf.xml    14 Nov 2005 00:21:40 -0000      1.120

+++ conf.xml    19 Nov 2005 22:54:02 -0000

@@ -232,7 +232,7 @@

        <configstring name="basedn" desc="The base DN for the LDAP server"/>

        <configstring name="binddn" required="false" desc="The DN used 
to bind to the LDAP server"/>

        <configstring name="password" required="false" desc="The 
password used to bind to the LDAP server"/>

-      <configenum name="version" desc="LDAP Protocol Version">2

+      <configenum name="version" desc="LDAP Protocol Version">3

         <values>

          <value desc="LDAPv2 (Deprecated)">2</value>

          <value desc="LDAPv3">3</value>

@@ -710,7 +710,7 @@

        <configsection name="params">

         <configstring name="hostspec" desc="The hostname of the LDAP 
server">localhost</configstring>

         <configinteger name="port" desc="The port of the LDAP 
server">389</configinteger>

-       <configenum name="version" desc="LDAP Protocol Version">2

+       <configenum name="version" desc="LDAP Protocol Version">3

          <values>

           <value desc="LDAPv2 (Deprecated)">2</value>

           <value desc="LDAPv3">3</value>




Saved Queries