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

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