Fix for debug messages always showing up

OpenLiberty · Apr 3, 2023 · f072ada · f072ada
1 parent f2409d5
commit f072ada
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 8 deletions.
diff --git a/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/ServerFeatureSupport.java b/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/ServerFeatureSupport.java
@@ -36,25 +36,31 @@ protected class ServerFeatureMojoUtil extends ServerFeatureUtil {
 
         @Override
         public void debug(String msg) {
-            log.debug(msg);
+            if (isDebugEnabled()) {
+                log.debug(msg);
+            }
         }
 
         @Override
         public void debug(String msg, Throwable e) {
-            log.debug(msg, e);
+            if (isDebugEnabled()) {
+                log.debug(msg, e);
+            }
         }
 
         @Override
         public void debug(Throwable e) {
-            log.debug(e);
+            if (isDebugEnabled()) {
+                log.debug(e);
+            }
         }
 
         @Override
         public void warn(String msg) {
             if (!suppressLogs) {
                 log.warn(msg);
             } else {
-                log.debug(msg);
+                debug(msg);
             }
         }
 
@@ -63,7 +69,7 @@ public void info(String msg) {
             if (!suppressLogs) {
                 log.info(msg);
             } else {
-                log.debug(msg);
+                debug(msg);
             }
         }
 

diff --git a/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/utils/CommonLogger.java b/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/utils/CommonLogger.java
@@ -32,17 +32,23 @@ public static CommonLogger getInstance() {
 
     @Override
     public void debug(String msg) {
-        getLog().debug(msg);
+        if (isDebugEnabled()) {
+            getLog().debug(msg);
+        }
     }
 
     @Override
     public void debug(String msg, Throwable e) {
-        getLog().debug(msg, e);
+        if (isDebugEnabled()) {
+            getLog().debug(msg, e);
+        }
     }
 
     @Override
     public void debug(Throwable e) {
-        getLog().debug(e);
+        if (isDebugEnabled()) {
+            getLog().debug(e);
+        }
     }
 
     @Override