From e320bc95f0a31bfbd408ef78e79d1127253d2d06 Mon Sep 17 00:00:00 2001 From: ruv Date: Sat, 28 Feb 2026 21:42:42 -0500 Subject: [PATCH] fix: Remove process.env reference from browser ES module process.env does not exist in vanilla browser ES modules (no bundler). Use window.location.protocol check only for WSS detection. Co-Authored-By: claude-flow --- ui/config/api.config.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ui/config/api.config.js b/ui/config/api.config.js index 1b14d10..5900dc0 100644 --- a/ui/config/api.config.js +++ b/ui/config/api.config.js @@ -107,12 +107,12 @@ export function buildApiUrl(endpoint, params = {}) { // Helper function to build WebSocket URLs export function buildWsUrl(endpoint, params = {}) { - // Always use secure WebSocket (wss://) in production or when using HTTPS + // Use secure WebSocket (wss://) when serving over HTTPS or on non-localhost // Use ws:// only for localhost development const isLocalhost = window.location.hostname === 'localhost' || window.location.hostname === '127.0.0.1'; - const isProduction = window.location.protocol === 'https:' || process.env.NODE_ENV === 'production'; - const protocol = (isProduction || !isLocalhost) - ? API_CONFIG.WSS_PREFIX + const isSecure = window.location.protocol === 'https:'; + const protocol = (isSecure || !isLocalhost) + ? API_CONFIG.WSS_PREFIX : API_CONFIG.WS_PREFIX; // Use localhost:8000 for WebSocket connections to match FastAPI backend