proto: mark nosemgrep warning

fogbender · Aug 29, 2023 · bb6b077 · bb6b077
1 parent 568ec86
commit bb6b077
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/.changeset/late-donuts-shave.md b/.changeset/late-donuts-shave.md
@@ -0,0 +1,5 @@
+---
+"fogbender-proto": patch
+---
+
+add nosemgrep comment to ignore issue that should not be affecting production builds
diff --git a/packages/fogbender-proto/src/config.ts b/packages/fogbender-proto/src/config.ts
@@ -34,6 +34,8 @@ export function getServerApiUrl(env?: Env, client?: Client) {
 
 export function getServerWsUrl(env?: Env, client?: Client) {
   const serverApiUrl = getServerApiUrl(env, client);
+  // we need to have "ws" support for localhost
+  // nosemgrep: javascript.lang.security.detect-insecure-websocket.detect-insecure-websocket
   const wsPath = serverApiUrl.replace("https://", "wss://").replace("http://", "ws://");
   return `${wsPath}/ws/v2`;
 }