diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-22 11:00:43 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2021-01-22 11:00:43 +0100 |
commit | 42f5e65a606ee01350179992a7cad0bda8891f3e (patch) | |
tree | d48b7360652e0f547cf62b4162631959caa62c41 /jdisc_http_service/src/main/java/com/yahoo | |
parent | e0efd6c132f799db8453e6052a728e435b87159c (diff) |
Handle null path (for invalid requests)
Diffstat (limited to 'jdisc_http_service/src/main/java/com/yahoo')
-rw-r--r-- | jdisc_http_service/src/main/java/com/yahoo/container/logging/JSONFormatter.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/jdisc_http_service/src/main/java/com/yahoo/container/logging/JSONFormatter.java b/jdisc_http_service/src/main/java/com/yahoo/container/logging/JSONFormatter.java index dc0acc4fbac..e702396b841 100644 --- a/jdisc_http_service/src/main/java/com/yahoo/container/logging/JSONFormatter.java +++ b/jdisc_http_service/src/main/java/com/yahoo/container/logging/JSONFormatter.java @@ -58,7 +58,7 @@ public class JSONFormatter { generator.writeNumberField("responsesize", entry.contentSize().orElse(0)); generator.writeNumberField("code", entry.statusCode().orElse(0)); generator.writeStringField("method", entry.httpMethod().orElse("")); - generator.writeStringField("uri", getNormalizedURI(entry.rawPath().get(), entry.rawQuery().orElse(null))); + generator.writeStringField("uri", getNormalizedURI(entry.rawPath().orElse(null), entry.rawQuery().orElse(null))); generator.writeStringField("version", entry.httpVersion().orElse("")); generator.writeStringField("agent", entry.userAgent().orElse("")); generator.writeStringField("host", entry.hostString().orElse("")); @@ -217,6 +217,7 @@ public class JSONFormatter { } private static String getNormalizedURI(String rawPath, String rawQuery) { + if (rawPath == null) return null; return rawQuery != null ? rawPath + "?" + rawQuery : rawPath; } |