CAS-1085

fix to both LDAP handlers to use the provided PasswordEncoder.
cyrille-leclerc · Jan 4, 2012 · 5b59f03 · 5b59f03
1 parent 1201406
commit 5b59f03
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...support-ldap/src/main/java/org/jasig/cas/adaptors/ldap/BindLdapAuthenticationHandler.java b/...support-ldap/src/main/java/org/jasig/cas/adaptors/ldap/BindLdapAuthenticationHandler.java
@@ -103,7 +103,7 @@ public void handleNameClassPair(final NameClassPair nameClassPair) {
                 this.log.debug("Performing LDAP bind with credential: " + dn);
                 test = this.getContextSource().getContext(
                     finalDn,
-                    credentials.getPassword());
+                    getPasswordEncoder().encode(credentials.getPassword()));
 
                 if (test != null) {
                     return true;

diff --git a/...ort-ldap/src/main/java/org/jasig/cas/adaptors/ldap/FastBindLdapAuthenticationHandler.java b/...ort-ldap/src/main/java/org/jasig/cas/adaptors/ldap/FastBindLdapAuthenticationHandler.java
@@ -29,7 +29,7 @@ protected boolean authenticateUsernamePasswordInternal(final UsernamePasswordCre
             final String transformedUsername = getPrincipalNameTransformer().transform(credentials.getUsername());
             final String bindDn = LdapUtils.getFilterWithValues(getFilter(), transformedUsername);
             this.log.debug("Performing LDAP bind with credential: " + bindDn);
-            dirContext = this.getContextSource().getContext(bindDn, credentials.getPassword());
+            dirContext = this.getContextSource().getContext(bindDn, getPasswordEncoder().encode(credentials.getPassword()));
             return true;
         } catch (final NamingException e) {
             log.debug(e.getMessage(), e);