[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-3946) LdapIdentityStore can not use backup server (failover)

[Raimund Hölle (JIRA)]

LdapIdentityStore can not use backup server (failover)
------------------------------------------------------

                 Key: JBSEAM-3946
                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3946
             Project: Seam
          Issue Type: Feature Request
          Components: Security
    Affects Versions: 2.1.1.GA, 2.1.1.CR2, 2.1.1.CR1, 2.1.0.SP1
            Reporter: Raimund Hölle


Since LdapIdentityStore assembles the provider URL internally, it is not possible to use extended URL's, e. g., a primary and a backup server like this:

env.setProperty(Context.PROVIDER_URL, "ldap://srv01:389 ldap://srv02:389");

It would be nice if we would be able to set the providerUrl directly. For compatibility reasons, i suggest the following change:

  // new property settable from components.xml
  private String providerUrl = null;

  protected final InitialLdapContext initialiseContext(String principal, String credentials) throws NamingException {
    ...
    String url = providerUrl;
    if (url == null || url.equals("")) {
      url = String.format("ldap://%s:%d", getServerAddress(), getServerPort());
    }
    env.setProperty(Context.PROVIDER_URL, url);
    ...

Additionally it would be nice if initialiseContext() wouldn't be final so we are able to extend the class.

Regards,
Raimund

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira