[jboss-cvs] JBossAS SVN: r108980 - projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit.

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

Author: darran.lofthouse at jboss.com
Date: 2010-10-29 04:57:56 -0400 (Fri, 29 Oct 2010)
New Revision: 108980

Modified:
   projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java
Log:
The test servlets are expected to be run in an environment with debug logging enabled so if looking for the optimum setting unless we are talking about creating a large log message log.isDebugEnabled messages should be ommitted as in the expected environment this will need to be evaluated twice - once for the log.isDebugEnabled call and once for the log.debug call.


Modified: projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java
===================================================================
--- projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java	2010-10-29 07:26:34 UTC (rev 108979)
+++ projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java	2010-10-29 08:57:56 UTC (rev 108980)
@@ -103,8 +103,7 @@
       try
       {
          LoginContext context = new LoginContext(securityDomain);
-         if (log.isDebugEnabled())
-            log.debug("Obtained LoginContext for '" + securityDomain + "' security-domain.");
+         log.debug("Obtained LoginContext for '" + securityDomain + "' security-domain.");
 
          context.login();
          writer.println("<h4>Authenticated</h4>");