[jboss-cvs] Picketbox SVN: r366 - in trunk: security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi and 2 other directories.

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

Author: sguilhen at redhat.com
Date: 2012-12-17 12:45:42 -0500 (Mon, 17 Dec 2012)
New Revision: 366

Modified:
   trunk/picketbox-infinispan/src/main/java/org/jboss/security/authentication/JBossCachedAuthenticationManager.java
   trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/AbstractServerLoginModule.java
   trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/plugins/auth/JaasSecurityManagerBase.java
   trunk/security-spi/common/src/main/java/org/jboss/security/PicketBoxLogger.java
Log:
SECURITY-703 Change log level of failed login attemps to DEBUG

Modified: trunk/picketbox-infinispan/src/main/java/org/jboss/security/authentication/JBossCachedAuthenticationManager.java
===================================================================
--- trunk/picketbox-infinispan/src/main/java/org/jboss/security/authentication/JBossCachedAuthenticationManager.java	2012-12-17 17:38:34 UTC (rev 365)
+++ trunk/picketbox-infinispan/src/main/java/org/jboss/security/authentication/JBossCachedAuthenticationManager.java	2012-12-17 17:45:42 UTC (rev 366)
@@ -367,7 +367,7 @@
 	   {
 		   // Don't log anonymous user failures unless trace level logging is on
 		   if (principal != null && principal.getName() != null)
-               PicketBoxLogger.LOGGER.errorDuringLogin(e);
+               PicketBoxLogger.LOGGER.debugFailedLogin(e);
 		   authException = e;
 	   }
 	   // Set the security association thread context info exception

Modified: trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/AbstractServerLoginModule.java
===================================================================
--- trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/AbstractServerLoginModule.java	2012-12-17 17:38:34 UTC (rev 365)
+++ trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/auth/spi/AbstractServerLoginModule.java	2012-12-17 17:45:42 UTC (rev 366)
@@ -196,7 +196,7 @@
          }
          catch(Exception e)
          {   // Dump the exception and continue
-            PicketBoxLogger.LOGGER.errorDuringLogin(e);
+            PicketBoxLogger.LOGGER.debugFailedLogin(e);
          }
       }
       return false;

Modified: trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/plugins/auth/JaasSecurityManagerBase.java
===================================================================
--- trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/plugins/auth/JaasSecurityManagerBase.java	2012-12-17 17:38:34 UTC (rev 365)
+++ trunk/security-jboss-sx/jbosssx/src/main/java/org/jboss/security/plugins/auth/JaasSecurityManagerBase.java	2012-12-17 17:45:42 UTC (rev 366)
@@ -346,7 +346,7 @@
 			// Don't log anonymous user failures unless trace level logging is
 			// on
 			if (principal != null && principal.getName() != null)
-                PicketBoxLogger.LOGGER.errorDuringLogin(e);
+                PicketBoxLogger.LOGGER.debugFailedLogin(e);
 			authException = e;
 		}
 		// Set the security association thread context info exception

Modified: trunk/security-spi/common/src/main/java/org/jboss/security/PicketBoxLogger.java
===================================================================
--- trunk/security-spi/common/src/main/java/org/jboss/security/PicketBoxLogger.java	2012-12-17 17:38:34 UTC (rev 365)
+++ trunk/security-spi/common/src/main/java/org/jboss/security/PicketBoxLogger.java	2012-12-17 17:45:42 UTC (rev 366)
@@ -43,9 +43,9 @@
     @Message(id = 205, value = "End validateCache, result = %s")
     void traceEndValidteCache(boolean isValid);
 
-    @LogMessage(level = Logger.Level.ERROR)
+    @LogMessage(level = Logger.Level.DEBUG)
     @Message(id = 206, value = "Login failure")
-    void errorDuringLogin(@Cause Throwable t);
+    void debugFailedLogin(@Cause Throwable t);
 
     @LogMessage(level = Logger.Level.TRACE)
     @Message(id = 207, value = "updateCache, input subject: %s, cached subject: %s")