[Jboss-cvs] JBossAS SVN: r56254 - trunk/security/src/main/org/jboss/security/auth/spi

[jboss-cvs-commits at lists.jboss.org]

Author: dimitris at jboss.org
Date: 2006-08-25 07:12:51 -0400 (Fri, 25 Aug 2006)
New Revision: 56254

Modified:
   trunk/security/src/main/org/jboss/security/auth/spi/LdapExtLoginModule.java
Log:
JBAS-3555, allow anonymous login with no bindDN and bindCredential

Modified: trunk/security/src/main/org/jboss/security/auth/spi/LdapExtLoginModule.java
===================================================================
--- trunk/security/src/main/org/jboss/security/auth/spi/LdapExtLoginModule.java	2006-08-25 10:53:54 UTC (rev 56253)
+++ trunk/security/src/main/org/jboss/security/auth/spi/LdapExtLoginModule.java	2006-08-25 11:12:51 UTC (rev 56254)
@@ -489,8 +489,11 @@
          providerURL = "ldap://localhost:" + ((protocol != null && protocol.equals("ssl")) ? "636" : "389");
 
       env.setProperty(Context.PROVIDER_URL, providerURL);
-      env.setProperty(Context.SECURITY_PRINCIPAL, dn);
-      env.put(Context.SECURITY_CREDENTIALS, credential);
+      // JBAS-3555, allow anonymous login with no bindDN and bindCredential
+      if (dn != null)
+         env.setProperty(Context.SECURITY_PRINCIPAL, dn);
+      if (credential != null)
+         env.put(Context.SECURITY_CREDENTIALS, credential);
       traceLdapEnv(env); 
       return new InitialLdapContext(env, null);
    }