summaryrefslogtreecommitdiffstats
path: root/container-core
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2019-12-13 11:24:48 +0100
committerBjørn Christian Seime <bjorncs@verizonmedia.com>2019-12-13 11:24:51 +0100
commitca585ef51aa1ffbe3510f44274dd4922713955dc (patch)
tree7beb2b474e2c9f6b910f6ef1f37de92a6ffbb75c /container-core
parent14b0a54720077edf95d270741d207f9015a1c7aa (diff)
Don't print full URI to log
The port printed is the local listen port, not the port used by client.
Diffstat (limited to 'container-core')
-rw-r--r--container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java12
1 files changed, 11 insertions, 1 deletions
diff --git a/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java b/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java
index 705092ef12e..488ce344f4a 100644
--- a/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java
+++ b/container-core/src/main/java/com/yahoo/container/jdisc/LoggingRequestHandler.java
@@ -240,7 +240,7 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler {
renderStartTime,
commitStartTime,
endTime,
- jdiscRequest.getUri().toString(),
+ getUri(jdiscRequest),
extendedResponse.getParsedQuery(),
extendedResponse.getTiming());
@@ -272,6 +272,16 @@ public abstract class LoggingRequestHandler extends ThreadedHttpRequestHandler {
accessLog.log(accessLogEntry);
}
+
+ private String getUri(com.yahoo.jdisc.http.HttpRequest jdiscRequest) {
+ URI uri = jdiscRequest.getUri();
+ StringBuilder builder = new StringBuilder(uri.getPath());
+ String query = uri.getQuery();
+ if (query != null && !query.isBlank()) {
+ builder.append('?').append(query);
+ }
+ return builder.toString();
+ }
}
private void populateAccessLogEntryNotCreatedByHttpServer(AccessLogEntry logEntry,