]
Shane Bryzak updated JBSEAM-3946:
---------------------------------
Fix Version/s: 3.0.0.BETA1
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.0.SP1, 2.1.1.CR1, 2.1.1.CR2, 2.1.1.GA
Reporter: Raimund Hölle
Assignee: Shane Bryzak
Fix For: 3.0.0.BETA1
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: