fix: Allow warn logging for requests which are failures, ensure debug…

… logging level for all other requests (DEV-1255)
dasch-swiss · Nov 2, 2022 · f482cf8 · f482cf8
1 parent b359916
commit f482cf8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/webapi/src/main/resources/logback.xml b/webapi/src/main/resources/logback.xml
@@ -38,6 +38,7 @@
     <logger name="org.knora.webapi.util.cache" level="INFO"/>
     <logger name="org.knora.webapi.util.PermissionUtilADM" level="INFO"/>
     <logger name="org.knora.webapi.update.UpdateRepository" level="INFO"/>
+    <logger name="org.knora.webapi.routing.AroundDirectives" level="INFO"/>
     <logger name="org.knora.webapi.routing.Authenticator" level="INFO"/>
     <logger name="org.knora.webapi.routing.JWTHelper" level="INFO"/>
     <logger name="org.knora.webapi.routing.v1.AssetsRouteV1" level="INFO"/>

diff --git a/webapi/src/main/scala/org/knora/webapi/routing/AroundDirectives.scala b/webapi/src/main/scala/org/knora/webapi/routing/AroundDirectives.scala
@@ -21,10 +21,9 @@ trait AroundDirectives extends InstrumentationSupport {
   def logDuration: Directive0 = extractRequestContext.flatMap { ctx =>
     val start = System.currentTimeMillis()
     mapResponse { resp =>
-      val took = System.currentTimeMillis() - start
-      metricsLogger.info(
-        s"[${resp.status.intValue()}] ${ctx.request.method.name} " + s"${ctx.request.uri} took: ${took}ms"
-      )
+      val took    = System.currentTimeMillis() - start
+      val message = s"[${resp.status.intValue()}] ${ctx.request.method.name} " + s"${ctx.request.uri} took: ${took}ms"
+      if (resp.status.isFailure()) metricsLogger.warn(message) else metricsLogger.debug(message)
       resp
     }
   }