Merge pull request #48796 from michalvavrik/3.20

jmartisk · web-flow · commit 99c4db2190be · 2025-07-07T07:36:39.000+02:00
[3.20] WebSockets Next: Make sure non-websocket connections are ignored
diff --git a/extensions/websockets-next/runtime/src/main/java/io/quarkus/websockets/next/runtime/WebSocketServerRecorder.java b/extensions/websockets-next/runtime/src/main/java/io/quarkus/websockets/next/runtime/WebSocketServerRecorder.java
@@ -86,6 +86,7 @@ public void handle(RoutingContext ctx) {
                 if (!ctx.request().headers().contains(HandshakeRequest.SEC_WEBSOCKET_KEY)) {
                     LOG.debugf("Non-websocket client request ignored:\n%s", ctx.request().headers());
                     ctx.next();
+                    return;
                 }
                 if (httpUpgradeChecks != null) {
                     checkHttpUpgrade(ctx, endpointId).subscribe().with(result -> {

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,7 @@ public void handle(RoutingContext ctx) {`
`86`	`86`	`if (!ctx.request().headers().contains(HandshakeRequest.SEC_WEBSOCKET_KEY)) {`
`87`	`87`	`LOG.debugf("Non-websocket client request ignored:\n%s", ctx.request().headers());`
`88`	`88`	`ctx.next();`
	`89`	`+ return;`
`89`	`90`	`}`
`90`	`91`	`if (httpUpgradeChecks != null) {`
`91`	`92`	`checkHttpUpgrade(ctx, endpointId).subscribe().with(result -> {`