Modify

Opened 7 years ago

Closed 5 years ago

#7873 closed enhancement (wontfix)

Add Compatibility for LDAP servers other than ActiveDirectory

Reported by: Vincent Owned by: John Hampton
Priority: normal Component: DirectoryAuthPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

I modified the plugin a bit to add some flexibility as to easily support openLDAP and other custom setups.

While my own setup could not work with the original plugin, the modifications I made should not break installed setups (except maybe because of default values in rare cases)

Please contact me at the email I used in the file headers.

Attachments (1)

experimental.zip (6.3 KB) - added by Vincent 7 years ago.
Experimental flexibility for openLDAP

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by Vincent

Attachment: experimental.zip added

Experimental flexibility for openLDAP

comment:1 Changed 7 years ago by John Hampton

Priority: highestnormal
Severity: blockernormal
Status: newassigned
Summary: I would ike to contribute to this pluginAdd Compatibility for LDAP servers other than ActiveDirectory

OK, first, this is neither high priority, nor is it a blocker of any kind.

I looked at the changes and they are reasonable. However, I have a few questions:

Why use the ActiveDirectoryAuthPlugin instead of one of the other LDAP auth plugins such as: TracLdapAuthPlugin or LdapAuthStorePlugin?

I'm not against modifying the ActiveDirectoryAuthPlugin, just seemed a bit of an interesting choice as it's clearly aimed at Active Directory rather than generic LDAP.

Expanding it beyond Active Directory also means that a name change may be in order. Unfortunately, with so may other "LDAP" plugins, just naming it "LDAP" something might not work. Name ideas? Perhaps simply clearer documentation?

Did you have any other ideas for the plugin?

comment:2 Changed 5 years ago by branson

Resolution: wontfix
Status: assignedclosed

No comments on this .. I think it's a 'wont'fix' If there's enough interest in this .. I may revisit/combine a few of the plugins mentioned to resolve this.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain John Hampton.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.