You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by ak...@apache.org on 2006/07/12 23:38:24 UTC
svn commit: r421422 - in /directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn: AuthenticationService.java SimpleAuthenticator.java

Author: akarasulu
Date: Wed Jul 12 14:38:23 2006
New Revision: 421422

URL: http://svn.apache.org/viewvc?rev=421422&view=rev
Log:
adding next interceptor calls to AuthenticationService which were missing :(

Modified:
    directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationService.java
    directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/SimpleAuthenticator.java

Modified: directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationService.java
URL: http://svn.apache.org/viewvc/directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationService.java?rev=421422&r1=421421&r2=421422&view=diff
==============================================================================
--- directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationService.java (original)
+++ directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationService.java Wed Jul 12 14:38:23 2006
@@ -421,6 +421,7 @@
             {
                 ctx.removeFromEnvironment( Context.SECURITY_CREDENTIALS );
             }
+            next.bind( bindDn, credentials, mechanisms, saslAuthId );
             return;
         }
 
@@ -447,6 +448,7 @@
                 .getAuthenticationLevel( ctx.getEnvironment() ) ) ) );
             // remove creds so there is no security risk
             ctx.removeFromEnvironment( Context.SECURITY_CREDENTIALS );
+            next.bind( bindDn, credentials, mechanisms, saslAuthId );
             return;
         }
 
@@ -462,6 +464,7 @@
                 ctx.setPrincipal( new TrustedPrincipalWrapper( authorizationId ) );
                 // remove creds so there is no security risk
                 ctx.removeFromEnvironment( Context.SECURITY_CREDENTIALS );
+                next.bind( bindDn, credentials, mechanisms, saslAuthId );
                 return;
             }
             catch ( LdapAuthenticationException e )

Modified: directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/SimpleAuthenticator.java
URL: http://svn.apache.org/viewvc/directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/SimpleAuthenticator.java?rev=421422&r1=421421&r2=421422&view=diff
==============================================================================
--- directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/SimpleAuthenticator.java (original)
+++ directory/branches/apacheds/optimization/core/src/main/java/org/apache/directory/server/core/authn/SimpleAuthenticator.java Wed Jul 12 14:38:23 2006
@@ -87,6 +87,11 @@
      */
     public LdapPrincipal authenticate( ServerContext ctx ) throws NamingException
     {
+        /*
+         * OPTIMIZE
+         * @TODO pass already normalized LdapDN in as an argument to authenticate
+         */
+        
         // ---- extract password from JNDI environment
 
         Object creds = ctx.getEnvironment().get( Context.SECURITY_CREDENTIALS );