Juster feillogging.

Author: chris-santa

src/main/kotlin/no/nav/tms/statistikk/statistikkApi.kt

@@ -23,14 +23,18 @@ fun Application.statistikkApi(
 
     install(StatusPages) {
         val log = KotlinLogging.logger {}
+        val securelog = KotlinLogging.logger("secureLog")
+
         exception<Throwable> { call, cause ->
             when (cause) {
                 is IllegalArgumentException -> {
-                    log.info { "Bad request til statistikkApi: $cause, ${cause.message.toString()}" }
+                    log.warn { "Bad request til statistikkApi" }
+                    securelog.warn(cause) { "Bad request til statistikkApi" }
                     call.respond(HttpStatusCode.BadRequest, cause.message ?: "")
                 }
                 else -> {
-                    log.error { "Feil i statistikkApi: $cause, ${cause.message.toString()}" }
+                    log.error { "Feil i statistikkApi" }
+                    securelog.error(cause) { "Feil i statistikkApi" }
                     call.respond(HttpStatusCode.InternalServerError)
                 }
             }

src/main/resources/logback.xml

@@ -1,10 +1,13 @@
 <configuration debug="true">
     <statusListener class="ch.qos.logback.core.status.OnConsoleStatusListener"/>
+
     <property name="loggingPattern"
               value="%d [%-5level] [%thread] %logger{5} %replace(- [%X{consumerId}, %X{callId}, %X{userId}] ){'- \[, , \] ',''}- %m%n"/>
+
     <appender name="stdout_json" class="ch.qos.logback.core.ConsoleAppender">
         <encoder class="net.logstash.logback.encoder.LogstashEncoder"/>
     </appender>
+
     <appender name="secureLog" class="ch.qos.logback.core.rolling.RollingFileAppender">
         <file>/secure-logs/secure.log</file>
         <rollingPolicy class="ch.qos.logback.core.rolling.FixedWindowRollingPolicy">
@@ -15,10 +18,13 @@
         <triggeringPolicy class="ch.qos.logback.core.rolling.SizeBasedTriggeringPolicy">
             <maxFileSize>50MB</maxFileSize>
         </triggeringPolicy>
-        <encoder class="net.logstash.logback.encoder.LogstashEncoder" />
+        <encoder class="net.logstash.logback.encoder.LogstashEncoder"/>
     </appender>
+
     <root level="INFO">
         <appender-ref ref="stdout_json"/>
-        <appender-ref ref="secureLog"/>
     </root>
-</configuration>
+    <logger name="secureLog" level="INFO" additivity="false">
+        <appender-ref ref="secureLog"/>
+    </logger>
+</configuration>