fix(gui): bind dev server to localhost by default, open binding in co…

…ntainers
ssube · Jan 8, 2023 · fc988e4 · fc988e4
1 parent 8c133e9
commit fc988e4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/gui/Containerfile.node.alpine b/gui/Containerfile.node.alpine
@@ -2,6 +2,8 @@ FROM docker.io/node:18-alpine
 
 ENV PATH="${PATH}:$(yarn global bin)"
 
+ENV ONNX_WEB_DEV_HOST="0.0.0.0"
+
 WORKDIR /onnx-web
 
 # copy package first, to invalidate other layers when version changes

diff --git a/gui/Containerfile.node.buster b/gui/Containerfile.node.buster
@@ -2,6 +2,8 @@ FROM docker.io/node:18-buster
 
 ENV PATH="${PATH}:$(yarn global bin)"
 
+ENV ONNX_WEB_DEV_HOST="0.0.0.0"
+
 WORKDIR /onnx-web
 
 # copy package first, to invalidate other layers when version changes

diff --git a/gui/serve.js b/gui/serve.js
@@ -5,7 +5,7 @@ import { join } from 'path';
 
 const { env } = process;
 
-const host = mustDefault(env.ONNX_WEB_DEV_HOST, '0.0.0.0');
+const host = mustDefault(env.ONNX_WEB_DEV_HOST, '127.0.0.1');
 const port = mustDefault(env.ONNX_WEB_DEV_PORT, '3000');
 const root = process.cwd();