Merge pull request #1097 from golemfactory/bugfix/JST-1092

fix(tcp-proxy): throw an error when users try to use proxy in browser

src/network/tcp-proxy.test.ts

@@ -0,0 +1,15 @@
+jest.mock("../shared/utils", () => ({
+  checkAndThrowUnsupportedInBrowserError: () => {
+    throw new GolemUserError("Not supported in browser");
+  },
+}));
+
+import { GolemUserError } from "../shared/error/golem-error";
+
+import { TcpProxy } from "./tcp-proxy";
+
+describe("TCP Proxy in browser", () => {
+  test("Uses the checkAndThrowUnsupportedInBrowserError util to throw when the function detects browser environment", () => {
+    expect(() => new TcpProxy("ws://fake.url", "fake-app-key")).toThrow("Not supported in browser");
+  });
+});

src/network/tcp-proxy.ts

@@ -1,7 +1,7 @@
 import net from "net";
 import { WebSocket } from "ws";
 import { EventEmitter } from "eventemitter3";
-import { defaultLogger, Logger } from "../shared/utils";
+import { checkAndThrowUnsupportedInBrowserError, defaultLogger, Logger } from "../shared/utils";
 import { Buffer } from "buffer";
 
 export interface TcpProxyEvents {
@@ -64,8 +64,10 @@ export class TcpProxy {
     /**
      * Additional options of the proxy
      */
-    options: Partial<TcpProxyOptions>,
+    options: Partial<TcpProxyOptions> = {},
   ) {
+    checkAndThrowUnsupportedInBrowserError("TCP Proxy");
+
     this.heartBeatSec = options.heartBeatSec ?? 10;
     this.logger = options.logger ? options.logger.child("tcp-proxy") : defaultLogger("tcp-proxy");