Upgraded ws to 8.17.1 and solved some issues (#2415)

* Upgraded ws to 8.17.1 / 7.5.10 * Updated `webpack.react.babel.ts` to fix an issue where webpack tries to bundle browser version of `ws`
Chia-Network · Jun 24, 2024 · c9f4e15 · c9f4e15
1 parent 79ae095
commit c9f4e15
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 10 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/gui/package.json b/packages/gui/package.json
@@ -96,7 +96,7 @@
     "unique-names-generator": "4.6.0",
     "validator": "13.11.0",
     "victory": "^36.6.11",
-    "ws": "8.4.2"
+    "ws": "8.17.1"
   },
   "devDependencies": {
     "@babel/core": "7.22.10",

diff --git a/packages/gui/webpack.react.babel.ts b/packages/gui/webpack.react.babel.ts
@@ -100,6 +100,10 @@ export default {
       '@mui/styled-engine': '@mui/styled-engine-sc',
       crypto: 'crypto-browserify',
       stream: 'stream-browserify',
+      // Use `ws` package for nodejs instead of `ws` package for browsers
+      // Without this alias, webpack will bundle `ws` package for browsers
+      // See https://github.com/Chia-Network/chia-blockchain-gui/pull/2413#issuecomment-2181012908
+      ws: require.resolve('ws'),
     },
   },
   optimization: {