summaryrefslogtreecommitdiffstats
path: root/container-core
diff options
context:
space:
mode:
authorJon Marius Venstad <jonmv@users.noreply.github.com>2021-04-28 17:59:45 +0200
committerGitHub <noreply@github.com>2021-04-28 17:59:45 +0200
commitce5788525748183f72ad50a0774ca94fe66cef6f (patch)
tree865847939054f0b09fba632ceb5e192a8b3e72a9 /container-core
parent555456814583553216b74c89c79570a32ede23bb (diff)
parent084bd8c298832a84d66b483d947d5a16f779cd90 (diff)
Merge pull request #17632 from vespa-engine/bjorncs/fix
Handle null argument in request callbacks
Diffstat (limited to 'container-core')
-rw-r--r--container-core/src/main/java/com/yahoo/jdisc/http/server/jetty/JettyConnectionLogger.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/container-core/src/main/java/com/yahoo/jdisc/http/server/jetty/JettyConnectionLogger.java b/container-core/src/main/java/com/yahoo/jdisc/http/server/jetty/JettyConnectionLogger.java
index e7cdb13425f..bbeccb61c8a 100644
--- a/container-core/src/main/java/com/yahoo/jdisc/http/server/jetty/JettyConnectionLogger.java
+++ b/container-core/src/main/java/com/yahoo/jdisc/http/server/jetty/JettyConnectionLogger.java
@@ -153,7 +153,8 @@ class JettyConnectionLogger extends AbstractLifeCycle implements Connection.List
public void onResponseBegin(Request request) {
handleListenerInvocation("HttpChannel.Listener", "onResponseBegin", "%h", List.of(request), () -> {
SocketChannelEndPoint endpoint = findUnderlyingSocketEndpoint(request.getHttpChannel().getEndPoint());
- ConnectionInfo info = Objects.requireNonNull(connectionInfo.get(IdentityKey.of(endpoint)));
+ ConnectionInfo info = connectionInfo.get(IdentityKey.of(endpoint));
+ if (info == null) return; // Connection closed before response started - observed during Jetty server shutdown
info.incrementResponses();
});
}