Issue #6696 - don't keep Request object in the WebSocketNegotiation a…

…fter upgrade Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
jetty · Sep 6, 2021 · d41a297 · d41a297
1 parent dff5e14
commit d41a297
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/...re-server/src/main/java/org/eclipse/jetty/websocket/core/server/WebSocketNegotiation.java b/...re-server/src/main/java/org/eclipse/jetty/websocket/core/server/WebSocketNegotiation.java
@@ -31,10 +31,10 @@
 
 public abstract class WebSocketNegotiation
 {
-    private final Request baseRequest;
     private final HttpServletRequest request;
     private final HttpServletResponse response;
     private final WebSocketComponents components;
+    private Request baseRequest;
     private String version;
     private List<ExtensionConfig> offeredExtensions;
     private List<ExtensionConfig> negotiatedExtensions;
@@ -54,6 +54,11 @@ public Request getBaseRequest()
         return baseRequest;
     }
 
+    public void setBaseRequest(Request baseRequest)
+    {
+        this.baseRequest = baseRequest;
+    }
+
     public HttpServletRequest getRequest()
     {
         return request;

diff --git a/...er/src/main/java/org/eclipse/jetty/websocket/core/server/internal/AbstractHandshaker.java b/...er/src/main/java/org/eclipse/jetty/websocket/core/server/internal/AbstractHandshaker.java
@@ -167,6 +167,7 @@ public boolean upgradeRequest(WebSocketNegotiator negotiator, HttpServletRequest
         // Save state from request/response and remove reference to the base request/response.
         upgradeRequest.upgrade();
         upgradeResponse.upgrade();
+        negotiation.setBaseRequest(null);
 
         if (LOG.isDebugEnabled())
             LOG.debug("upgrade connection={} session={} framehandler={}", connection, coreSession, handler);