6.0.0-alpha12
6/8/25

[#1814] Improving LDAP support
Summary Improving LDAP support
Queue Turba
Queue Version FRAMEWORK_3
Type Bug
State Resolved
Priority 2. Medium
Owners
Requester ben.sommer (at) enc (dot) edu
Created 04/18/2005 (7356 days ago)
Due
Updated 05/24/2005 (7320 days ago)
Assigned
Resolved 05/24/2005 (7320 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
05/24/2005 09:46:28 PM Chuck Hagenbuch Comment #4
State ⇒ Resolved
Reply to this comment
I believe the fix for bug 2027 covers what Mike is talking about, in 
that the UID can be used in the DN. Given that I'm closing this ticket 
since I believe what needs to be addressed is addressed. More can even 
probably be done with hooks if necessary.
04/20/2005 11:01:41 PM mbydalek (at) compunetconsulting (dot) com Comment #3 Reply to this comment
Hi Ben,



I remember reading your initial posting to the list a week ago, and 
something didn't sit right with me in terms of what you were saying.   
I partially agreed, but partially didn't, and finally I had some time 
to sit down and look at some code.



Basically, I agree with what you are saying about having a 'cn' as the 
RDN, it just won't work.  But, there already is a generateUID() 
function which can be used.  Rather than modifying all the existing 
code, just changing the order of add() in Driver.php should do the 
trick.



I'm not sure how this would affect other drivers, but if you can get 
in $attributes the 'dn' (incorrectly named, should be 'rdn' as you 
pointed out) for when you do the _makeKey() call, you could use that 
to set your dn giving you a perfectly unique entry each time.



I hope some of this makes sense as it does get confusing after looking 
at it for a while ;)



-Mike
04/18/2005 07:53:29 PM ben (dot) sommer (at) enc (dot) edu Comment #2
New Attachment: sources.php.dist.diff Download
Reply to this comment
...and the second file...
04/18/2005 07:52:52 PM ben (dot) sommer (at) enc (dot) edu Comment #1
State ⇒ Unconfirmed
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Improving LDAP support
Queue ⇒ Turba
New Attachment: ldap.php.diff Download
Reply to this comment
This patch corrects an assumption about LDAP in turba that an RDN can 
be multi-valued. It also adds support for specifying an auto-generated 
value for an addressbook entry;s RDN. This is done via an anonynmous 
function created and configured in 'config/sources.php'. Code in 
lib/Driver/ldap.php that is no longer is needed after this patch was 
removed.

Saved Queries