diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-12-13 11:24:48 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2019-12-13 11:24:51 +0100 |
commit | ca585ef51aa1ffbe3510f44274dd4922713955dc (patch) | |
tree | 7beb2b474e2c9f6b910f6ef1f37de92a6ffbb75c /container-core | |
parent | 14b0a54720077edf95d270741d207f9015a1c7aa (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.java | 12 |
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, |