vttranlina commented on code in PR #1649:
URL: https://github.com/apache/james-project/pull/1649#discussion_r1268812105
##########
server/data/data-ldap/src/main/java/org/apache/james/user/ldap/LdapRepositoryConfiguration.java:
##########
@@ -202,13 +207,14 @@ public static LdapRepositoryConfiguration
from(HierarchicalConfiguration<Immutab
}
/**
- * The URL of the LDAP server against which users are to be authenticated.
+ * The list of URLs of the LDAP servers against which users are to be
authenticated.
* Note that users are actually authenticated by binding against the LDAP
- * server using the users "dn" and "credentials".The
+ * servers using the users "dn" and "credentials".The
* value of this field is taken from the value of the configuration
- * attribute "ldapHost".
+ * attribute "ldapHosts" and fallback to the legacy attribute
"ldapHost".
+ * URLs are split by the comma "," character.
*/
- private final String ldapHost;
+ private final List<String> ldapHosts;
Review Comment:
or `List<URI> ldapHosts`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
