fix(node-ws): websocket upgrade header value case sensitivity (#876)

honojs · Dec 10, 2024 · a092ffa · a092ffa
1 parent 6524a92
commit a092ffa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/cool-coats-try.md b/.changeset/cool-coats-try.md
@@ -0,0 +1,5 @@
+---
+'@hono/node-ws': patch
+---
+
+Fixed case-sensitivity of the WebSocket "Upgrade" header value
diff --git a/packages/node-ws/src/index.ts b/packages/node-ws/src/index.ts
@@ -63,7 +63,7 @@ export const createNodeWebSocket = (init: NodeWebSocketInit): NodeWebSocket => {
     },
     upgradeWebSocket: (createEvents) =>
       async function upgradeWebSocket(c, next) {
-        if (c.req.header('upgrade') !== 'websocket') {
+        if (c.req.header('upgrade')?.toLowerCase() !== 'websocket') {
           // Not websocket
           await next()
           return