aboutsummaryrefslogtreecommitdiffstats
path: root/jdisc_http_service/src/main/java/com/yahoo
diff options
context:
space:
mode:
authorBjørn Christian Seime <bjorncs@verizonmedia.com>2021-01-22 11:00:43 +0100
committerBjørn Christian Seime <bjorncs@verizonmedia.com>2021-01-22 11:00:43 +0100
commit42f5e65a606ee01350179992a7cad0bda8891f3e (patch)
treed48b7360652e0f547cf62b4162631959caa62c41 /jdisc_http_service/src/main/java/com/yahoo
parente0efd6c132f799db8453e6052a728e435b87159c (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.java3
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;
}