feat(web): Include requestUserAgent and requestPort in request log (

#843) This will make it easier to track down particular client libraries when multiple connectors are being used.
spinnaker · Jul 2, 2019 · 85341ee · 85341ee
1 parent a7952d7
commit 85341ee
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/...eb/src/main/groovy/com/netflix/spinnaker/gate/interceptors/RequestLoggingInterceptor.java b/...eb/src/main/groovy/com/netflix/spinnaker/gate/interceptors/RequestLoggingInterceptor.java
@@ -44,6 +44,9 @@ public void afterCompletion(
     // 127.0.0.1 "GET /limecat.jpg HTTP/1.0" 200 2326
     try {
       MDC.put("requestDuration", getRequestDuration(request));
+      MDC.put("requestUserAgent", request.getHeader("User-Agent"));
+      MDC.put("requestPort", String.valueOf(request.getServerPort()));
+
       log.debug(
           "{} \"{} {} {}\" {} {}",
           value("sourceIp", sourceIpAddress(request)),
@@ -54,6 +57,8 @@ public void afterCompletion(
           value("responseSize", getResponseSize(response)));
     } finally {
       MDC.remove("requestDuration");
+      MDC.remove("requestUserAgent");
+      MDC.remove("requestPort");
     }
   }