diff options
author | Vlad Uspensky <vlad.uspensky@dominodatalab.com> | 2018-06-25 17:09:43 -0700 |
---|---|---|
committer | Zach Smith <zach@driver.xyz> | 2018-06-25 17:09:43 -0700 |
commit | 981cc63b94c6df5fc8f4d2f6ebafd1a6f27d7c4e (patch) | |
tree | 1ef2ba11719123bf0433dcd9fdb3539cd40deebc /src/main/scala/xyz/driver | |
parent | e292a16a560e6cfec2bf0d1afbd88c3277d21065 (diff) | |
download | driver-core-981cc63b94c6df5fc8f4d2f6ebafd1a6f27d7c4e.tar.gz driver-core-981cc63b94c6df5fc8f4d2f6ebafd1a6f27d7c4e.tar.bz2 driver-core-981cc63b94c6df5fc8f4d2f6ebafd1a6f27d7c4e.zip |
Info level short logging for requests and responses our services receive (#165)
* Info level short logging for requests and responses our services receive
* Remove STDOUT_DEBUG from deployed-logback.xml
Diffstat (limited to 'src/main/scala/xyz/driver')
-rw-r--r-- | src/main/scala/xyz/driver/core/app/DriverApp.scala | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/src/main/scala/xyz/driver/core/app/DriverApp.scala b/src/main/scala/xyz/driver/core/app/DriverApp.scala index 56ee898..6dd98e3 100644 --- a/src/main/scala/xyz/driver/core/app/DriverApp.scala +++ b/src/main/scala/xyz/driver/core/app/DriverApp.scala @@ -139,7 +139,7 @@ class DriverApp( storeRequestContextToMdc(ctx.request, origin, ip) - log.debug(s"""Received request {"method":"${ctx.request.method.value}","url": "${ctx.request.uri}"}""") + log.info(s"""Received request ${ctx.request.method.value} ${ctx.request.uri} (trace: $trackingId)""") val contextWithTrackingId = ctx.withRequest( @@ -147,9 +147,20 @@ class DriverApp( .addHeader(RawHeader(ContextHeaders.TrackingIdHeader, trackingId)) .addHeader(RawHeader(ContextHeaders.StacktraceHeader, updatedStacktrace))) - respondWithAllCorsHeaders { - combinedRoute - }(contextWithTrackingId) + val logResponses = mapRouteResult { + case c @ Complete(response) => + log.info( + s"Responded to ${ctx.request.method.value} ${ctx.request.uri} " + + s"with ${response.status.toString} (trace: $trackingId)") + c + case r @ Rejected(rejections) => + log.warn( + s"Request ${ctx.request.method.value} ${ctx.request.uri} " + + s"(trace: $trackingId) is rejected:\n${rejections.mkString(",\n")}") + r + } + + respondWithAllCorsHeaders(logResponses(combinedRoute))(contextWithTrackingId) } } |