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 2008/06/06 07:09:47 UTC
svn commit: r663825 -
/directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java
Author: akarasulu
Date: Thu Jun 5 22:09:47 2008
New Revision: 663825
URL: http://svn.apache.org/viewvc?rev=663825&view=rev
Log:
Fixing unnecessary log lines
Modified:
directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java
Modified: directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java?rev=663825&r1=663824&r2=663825&view=diff
==============================================================================
--- directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java (original)
+++ directory/apacheds/branches/bigbang/core/src/main/java/org/apache/directory/server/core/authn/AuthenticationInterceptor.java Thu Jun 5 22:09:47 2008
@@ -25,7 +25,6 @@
import java.util.HashMap;
import java.util.HashSet;
import java.util.Iterator;
-import java.util.List;
import java.util.Map;
import java.util.Set;
@@ -56,7 +55,7 @@
import org.apache.directory.shared.ldap.constants.AuthenticationLevel;
import org.apache.directory.shared.ldap.exception.LdapAuthenticationException;
import org.apache.directory.shared.ldap.name.LdapDN;
-import org.apache.directory.shared.ldap.util.StringTools;
+
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -202,7 +201,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Adding the entry {} for DN = '{}'", opContext.getEntry(), opContext.getDn().getUpName() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -214,7 +213,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Deleting name = '{}'", opContext.getDn().getUpName() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -227,7 +226,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Matching name = '{}'", opContext.getDn().getUpName() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -239,7 +238,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Getting root DSE" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -251,7 +250,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Getting suffix for name = '{}'", opContext.getDn().getUpName() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -263,7 +262,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Testing if entry name = '{}' exists", opContext.getDn().getUpName() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -275,7 +274,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Listing base = '" + opContext.getDn().getUpName() + "'" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -287,7 +286,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Listing suffixes" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -299,16 +298,7 @@
{
if ( IS_DEBUG )
{
- List<String> attrIds = opContext.getAttrsId();
-
- if ( ( attrIds != null ) && ( attrIds.size() != 0 ) )
- {
- String attrs = StringTools.listToString( attrIds );
- LOG.debug( "Lookup name = '" + opContext.getDn().getUpName() + "', attributes = " + attrs );
- } else
- {
- LOG.debug( "Lookup name = '" + opContext.getDn().getUpName() + "', no attributes " );
- }
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -335,7 +325,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( opContext.toString() );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -348,9 +338,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Modifying name = '" + opContext.getDn().getUpName() + "', new RDN = '" +
- opContext.getNewRdn() + "', " +
- "oldRDN = '" + opContext.getDelOldDn() + "'" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -364,10 +352,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Moving name = '" + opContext.getDn().getUpName() + "' to name = '" +
- opContext.getParent() + "', new RDN = '" +
- opContext.getNewRdn() + "', oldRDN = '" +
- opContext.getDelOldDn() + "'" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -380,8 +365,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Moving name = '" + opContext.getDn().getUpName() + " to name = '" +
- opContext.getParent().getUpName() + "'" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -394,7 +378,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "Search for base = '" + opContext.getDn().getUpName() + "'" );
+ LOG.debug( "Operation Context: {}", opContext );
}
checkAuthenticated( opContext );
@@ -423,7 +407,7 @@
{
if ( IS_DEBUG )
{
- LOG.debug( "bind: principal: " + opContext.getDn() );
+ LOG.debug( "Operation Context: {}", opContext );
}
if ( opContext.getSession() != null && opContext.getSession().getEffectivePrincipal() != null )