Modify

Opened 4 years ago

Closed 2 years ago

#7873 closed enhancement (wontfix)

Add Compatibility for LDAP servers other than ActiveDirectory

Reported by: Naerymdan Owned by: pacopablo
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 Naerymdan 4 years ago.
Experimental flexibility for openLDAP

Download all attachments as: .zip

Change History (3)

Changed 4 years ago by Naerymdan

Experimental flexibility for openLDAP

comment:1 Changed 4 years ago by pacopablo

  • Priority changed from highest to normal
  • Severity changed from blocker to normal
  • Status changed from new to assigned
  • Summary changed from I would ike to contribute to this plugin to Add 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 2 years ago by sandinak

  • Resolution set to wontfix
  • Status changed from assigned to closed

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.

Add Comment

Modify Ticket

Action
as closed The owner will remain pacopablo.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.