Add support for ldaps:// urls. (Fix for SEC-146).

This commit is contained in:
Luke Taylor 2006-01-27 18:28:13 +00:00
parent 82be52cea0
commit fbe5957c23
2 changed files with 7 additions and 1 deletions

View File

@ -129,7 +129,7 @@ public class DefaultInitialDirContextFactory implements InitialDirContextFactory
Assert.hasLength(url, "An LDAP connection URL must be supplied.");
if (url.startsWith("ldap:")) {
if (url.startsWith("ldap:") || url.startsWith("ldaps:")) {
URI uri = LdapUtils.parseLdapUrl(url);

View File

@ -40,6 +40,12 @@ public class DefaultInitialDirContextFactoryTests extends AbstractLdapServerTest
}
public void testSecureLdapUrlIsSupported() {
idf = new DefaultInitialDirContextFactory("ldaps://localhost/dc=acegisecurity,dc=org");
assertEquals("dc=acegisecurity,dc=org", idf.getRootDn());
}
public void testConnectionFailure() throws Exception {
// Use the wrong port
idf = new DefaultInitialDirContextFactory("ldap://localhost:60389");