avoid new string, connection must be upgrade for websocket

robaho · robaho · commit 5a7289b4e269 · 2024-07-04T10:04:56.000-05:00
diff --git a/src/main/java/robaho/net/httpserver/websockets/WebSocketHandler.java b/src/main/java/robaho/net/httpserver/websockets/WebSocketHandler.java
@@ -59,7 +59,7 @@ public void handle(HttpExchange exchange) throws IOException {
     public static boolean isWebsocketRequested(Headers headers) {
         // check if Upgrade connection
         String connection = headers.getFirst(Util.HEADER_CONNECTION);
-        if (connection == null || !connection.toLowerCase().contains(Util.HEADER_CONNECTION_VALUE.toLowerCase())) {
+        if (connection == null || !connection.equalsIgnoreCase(Util.HEADER_CONNECTION_VALUE)) {
             return false;
         }
         // check for proper upgrade tyoe

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public void handle(HttpExchange exchange) throws IOException {`
`59`	`59`	`public static boolean isWebsocketRequested(Headers headers) {`
`60`	`60`	`// check if Upgrade connection`
`61`	`61`	`String connection = headers.getFirst(Util.HEADER_CONNECTION);`
`62`		`- if (connection == null \|\| !connection.toLowerCase().contains(Util.HEADER_CONNECTION_VALUE.toLowerCase())) {`
	`62`	`+ if (connection == null \|\| !connection.equalsIgnoreCase(Util.HEADER_CONNECTION_VALUE)) {`
`63`	`63`	`return false;`
`64`	`64`	`}`
`65`	`65`	`// check for proper upgrade tyoe`