]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-7500 TomcatAccessLog must log at DEBUG level
authorSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Mon, 20 Feb 2017 17:27:42 +0000 (18:27 +0100)
committerSébastien Lesaint <sebastien.lesaint@sonarsource.com>
Fri, 24 Feb 2017 19:27:07 +0000 (20:27 +0100)
the information that the web server is started/stopped is already logged by other classes with logger name more relevant to the labels of the logs

server/sonar-server/src/main/java/org/sonar/server/app/TomcatAccessLog.java
server/sonar-server/src/test/java/org/sonar/server/app/TomcatAccessLogTest.java

index fcd441d8122eccf0ab03e6b3a69bbe6640c27157..a51db1fbe15879ba06920378f7421219d7086ef1 100644 (file)
@@ -68,10 +68,10 @@ class TomcatAccessLog {
     @Override
     public void lifecycleEvent(LifecycleEvent event) {
       if ("after_start".equals(event.getType())) {
-        logger.info("Web server is started");
+        logger.debug("Tomcat is started");
 
       } else if ("after_destroy".equals(event.getType())) {
-        logger.info("Web server is stopped");
+        logger.debug("Tomcat is stopped");
       }
     }
   }
index d63a6a3cc1db2339f84f0592de96b232b303f43d..0d4f290ce3610061f8f9e407545d62808b67c281 100644 (file)
@@ -71,10 +71,10 @@ public class TomcatAccessLogTest {
 
     event = new LifecycleEvent(mock(Lifecycle.class), "after_start", null);
     listener.lifecycleEvent(event);
-    verify(logger).info("Web server is started");
+    verify(logger).debug("Tomcat is started");
 
     event = new LifecycleEvent(mock(Lifecycle.class), "after_destroy", null);
     listener.lifecycleEvent(event);
-    verify(logger).info("Web server is stopped");
+    verify(logger).debug("Tomcat is stopped");
   }
 }