6.0.0-beta1
7/8/25

[#8433] Sample for Active Directory server and more comments
Summary Sample for Active Directory server and more comments
Queue Turba
Queue Version 2.3.1
Type Enhancement
State Resolved
Priority 2. Medium
Owners
Requester citycat4 (at) ngs (dot) ru
Created 07/14/2009 (5838 days ago)
Due
Updated 03/19/2010 (5590 days ago)
Assigned
Resolved 08/05/2009 (5816 days ago)
Milestone 2.3.2
Patch Yes

History
03/19/2010 01:59:34 AM coolantro_02 (at) yahoo (dot) com Comment #11 Reply to this comment
Are there so many changed compared to the 'localldap' source that 
you can't provide the settings for AD as alternatives in that 
configuration? I'd prefer that over a complete new source entry 
that's basically the same.
01/13/2010 12:10:02 AM CVS Commit Comment #10 Reply to this comment
Changes have been made in Git for this ticket:

Improve LDAP documentation a lot. Add special configuration 
instructions for AD. Bug: 8433 Author: citycat4@ngs.ru

http://git.horde.org/diff.php/turba/config/sources.php.dist?rt=horde-git&r1=d58245fa6817cbbf97fc5169615794d0232ff08f&r2=d1d795fd67fd3805d43d34c8a0cf237bfdc4ac91
08/05/2009 09:05:41 PM Jan Schneider Comment #9 Reply to this comment
Fixed
08/05/2009 05:36:05 PM citycat4 (at) ngs (dot) ru Comment #8 Reply to this comment
Tweaked and commited. Basically I removed the redundant inline
documentation and fixed some formatting and wording.
Thanks a lot!
I have found one mistake



+//         // For Active Directory usually domain name:

+//         // 'root' => 'example.com',



That's incorrect. 'example.com' was only as sample, which you can 
obtain domain name. You should specify basedn (root) as usually 
'dc=example,dc=com'
08/05/2009 04:33:15 PM Jan Schneider Comment #7
Milestone ⇒ 2.3.2
State ⇒ Resolved
Reply to this comment
Tweaked and commited. Basically I removed the redundant inline 
documentation and fixed some formatting and wording.

Thanks a lot!
08/05/2009 04:32:06 PM Jan Schneider Deleted Original Message
 
08/05/2009 04:30:52 AM citycat4 (at) ngs (dot) ru Comment #5
New Attachment: sources.php.dist[1].patch Download
Reply to this comment
Reworked patch. Also I have added description of 'sizelimit' parameter
08/03/2009 09:47:33 PM Jan Schneider Comment #4 Reply to this comment
No, I don't want another source. Just add comments with the correct 
settings to the existing ldap source configuration.
08/03/2009 06:07:23 PM citycat4 (at) ngs (dot) ru Comment #3 Reply to this comment
Are there so many changed compared to the 'localldap' source that you
can't provide the settings for AD as alternatives in that
configuration? I'd prefer that over a complete new source entry
that's basically the same.
Well, I'll do it. It will be "localad" source. Patch will followed soon


07/20/2009 03:33:42 PM Jan Schneider Comment #2
State ⇒ Feedback
Reply to this comment
Are there so many changed compared to the 'localldap' source that you 
can't provide the settings for AD as alternatives in that 
configuration? I'd prefer that over a complete new source entry that's 
basically the same.
07/14/2009 11:01:03 AM citycat4 (at) ngs (dot) ru Comment #1
Priority ⇒ 2. Medium
State ⇒ New
New Attachment: sources.php.dist.patch
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Turba
Summary ⇒ Sample for Active Directory server and more comments
Type ⇒ Enhancement
Reply to this comment
I have done separate sapmle section for Active Directory server (based 
on my own) and add some more descriptions for parameters, which were 
missed in sources.php.dist, i.e. 'root', 'bind_dn', 'bind_password', 
'version'. Also I have include new parameter 'referrals' (described in 
bug #8432). This parameter required for Windows 2003 server or high.

Saved Queries