From 5479ed030f753d7f2e494da73c087970aca2f06b Mon Sep 17 00:00:00 2001 From: Joakim Erdfelt Date: Wed, 13 Nov 2019 15:33:00 -0600 Subject: [PATCH] Issue #4248 - Using ServletUpgradeResponse.setSuccess(true) for servlet side Signed-off-by: Joakim Erdfelt --- .../eclipse/jetty/websocket/server/WebSocketServerFactory.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/jetty-websocket/websocket-server/src/main/java/org/eclipse/jetty/websocket/server/WebSocketServerFactory.java b/jetty-websocket/websocket-server/src/main/java/org/eclipse/jetty/websocket/server/WebSocketServerFactory.java index 2b5e116bd08..150db502c71 100644 --- a/jetty-websocket/websocket-server/src/main/java/org/eclipse/jetty/websocket/server/WebSocketServerFactory.java +++ b/jetty-websocket/websocket-server/src/main/java/org/eclipse/jetty/websocket/server/WebSocketServerFactory.java @@ -698,6 +698,8 @@ public class WebSocketServerFactory extends ContainerLifeCycle implements WebSoc // Process (version specific) handshake response handshaker.doHandshakeResponse(request, response); + response.setSuccess(true); + if (LOG.isDebugEnabled()) LOG.debug("Websocket upgrade {} {} {} {}", request.getRequestURI(), version, response.getAcceptedSubProtocol(), wsConnection);