[jboss-cvs] JBossAS SVN: r108969 - in projects/security/security-negotiation/trunk: jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit and 1 other directory.

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

Author: mmoyses
Date: 2010-10-28 10:40:48 -0400 (Thu, 28 Oct 2010)
New Revision: 108969

Modified:
   projects/security/security-negotiation/trunk/jboss-negotiation-common/src/main/java/org/jboss/security/negotiation/MessageFactory.java
   projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java
Log:
adding logging checks where string concatenation happens

Modified: projects/security/security-negotiation/trunk/jboss-negotiation-common/src/main/java/org/jboss/security/negotiation/MessageFactory.java
===================================================================
--- projects/security/security-negotiation/trunk/jboss-negotiation-common/src/main/java/org/jboss/security/negotiation/MessageFactory.java	2010-10-28 14:35:30 UTC (rev 108968)
+++ projects/security/security-negotiation/trunk/jboss-negotiation-common/src/main/java/org/jboss/security/negotiation/MessageFactory.java	2010-10-28 14:40:48 UTC (rev 108969)
@@ -65,7 +65,8 @@
       }
       catch (Exception e)
       {
-         log.trace("Unable to load class '" + classname + "'", e);
+         if (log.isTraceEnabled())
+            log.trace("Unable to load class '" + classname + "'", e);
       }
 
       return clazz;

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-28 14:35:30 UTC (rev 108968)
+++ projects/security/security-negotiation/trunk/jboss-negotiation-toolkit/src/main/java/org/jboss/security/negotiation/toolkit/SecurityDomainTestServlet.java	2010-10-28 14:40:48 UTC (rev 108969)
@@ -103,7 +103,8 @@
       try
       {
          LoginContext context = new LoginContext(securityDomain);
-         log.debug("Obtained LoginContext for '" + securityDomain + "' security-domain.");
+         if (log.isDebugEnabled())
+            log.debug("Obtained LoginContext for '" + securityDomain + "' security-domain.");
 
          context.login();
          writer.println("<h4>Authenticated</h4>");