WELD-1948 Bean store leak - build bean identifiers string only if debug

level is enabled
weld · Jul 21, 2015 · 611cb72 · 611cb72
1 parent 8362296
commit 611cb72
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/impl/src/main/java/org/jboss/weld/context/http/HttpRequestContextImpl.java b/impl/src/main/java/org/jboss/weld/context/http/HttpRequestContextImpl.java
@@ -54,7 +54,9 @@ public boolean associate(HttpServletRequest request) {
         BoundBeanStore beanStore = getBeanStore();
         if (beanStore != null) {
             ContextLogger.LOG.beanStoreLeakDuringAssociation(this.getClass().getName(), request);
-            ContextLogger.LOG.beanStoreLeakAffectedBeanIdentifiers(this.getClass().getName(), Iterables.toMultiRowString(beanStore));
+            if (ContextLogger.LOG.isDebugEnabled()) {
+                ContextLogger.LOG.beanStoreLeakAffectedBeanIdentifiers(this.getClass().getName(), Iterables.toMultiRowString(beanStore));
+            }
         }
         // We always associate a new bean store to avoid possible leaks (security threats)
         beanStore = new RequestBeanStore(request, namingScheme);