SEC-1182: allow multiple ldap-authentication-provider elements (or allow multiple user search bases) #1430

Closed
spring-issuemaster opened this Issue Jun 16, 2009 · 3 comments

1 participant

@spring-issuemaster

Karl Palsson (Migrated from SEC-1182) said:

As in the forum posting, but I can't find any ticket that "phamlen" created for this.

I'm ok with either the complete duplicate element syntax, as in the forum posting, or having just one provider and a list of search bases.

I'd probably go with the duplicate listing, as that falls in nicely with the other auth providers.

@spring-issuemaster

Karl Palsson said:

Patches for trunk and the 2.x branch. Makes "id" an allowed attribute on the ldap-authentication-provider elements, and registers the beans correctly using that ID. Existing auth code was already quite happy to have multiple auth sources. (This could apparently be done using direct bean config, just not using the schema config)

@spring-issuemaster

Luke Taylor said:

I think this should no longer be required, as of the changes for SEC-1196.

The global identifier for the LDAP provider no longer exists and all the namespace providers must be registered within the element. The beans will be parsed immediately and added to the provider list. So it should be possible to use multiple LDAP providers (though I haven't tried it yet :) ).

@spring-issuemaster

Luke Taylor said:

I've added a test to check that it is now possible to have two LDAP providers configured within the element, so closing the issue.

Thanks for taking the time to submit a patch though :).

@spring-issuemaster spring-issuemaster added this to the 3.0.0 M2 milestone Feb 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment