Merge pull request #205 from WalletConnect/fix/remove-mobile-headers

fix: remove socket mobile headers
WalletConnect · Dec 4, 2024 · c44fd04 · c44fd04
2 parents fbd47cd + ebca94c
commit c44fd04
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/jsonrpc/ws-connection/src/ws.ts b/jsonrpc/ws-connection/src/ws.ts
@@ -117,11 +117,7 @@ export class WsConnection implements IJsonRpcConnection {
     this.registering = true;
 
     return new Promise((resolve, reject) => {
-      const packageName = new URLSearchParams(url).get("packageName");
-      const bundleId = new URLSearchParams(url).get("bundleId");
-      const opts = isReactNative()
-        ? { headers: { bundleId, packageName } }
-        : { rejectUnauthorized: !isLocalhostUrl(url) };
+      const opts = !isReactNative() ? { rejectUnauthorized: !isLocalhostUrl(url) } : undefined;
       const socket: WebSocket = new WS(url, [], opts);
       if (hasBuiltInWebSocket()) {
         socket.onerror = (event: Event) => {