fix: ensure protocolVersion is a number in WebSocket options (#6013)

Author: barelyhuman

packages/bruno-requests/src/ws/ws-client.js

@@ -67,7 +67,11 @@ class WsClient {
       };
 
       if (protocolVersion) {
-        wsOptions.protocolVersion = protocolVersion;
+        // Force convert to number since `ws` doesn't do it for you
+        const asNumber = Number(protocolVersion);
+        if (!isNaN(asNumber)) {
+          wsOptions.protocolVersion = asNumber;
+        }
       }
 
       const wsConnection = new ws.WebSocket(parsedUrl.fullUrl, protocols, wsOptions);

tests/websockets/fixtures/collection/ws-test-request-with-subproto.bru

@@ -13,6 +13,7 @@ ws {
 headers {
   Sec-WebSocket-Protocol: soap
   Sec-WebSocket-Protocol: mqtt
+  Sec-WebSocket-Version: 13
 }
 
 body:ws {