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

[Shane Bryzak (JIRA)]

     [ https://jira.jboss.org/jira/browse/JBSEAM-3946?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

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: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira