From 9e9419bf21df5d366e17cb0a96c224ba3e277813 Mon Sep 17 00:00:00 2001
From: Felix Weinberger <fweinberger@anthropic.com>
Date: Thu, 26 Jun 2025 10:33:58 +0100
Subject: [PATCH] fix: ensure MCP_PROXY_PORT is included when auth is disabled

Address PR feedback by restructuring getClientUrl to always include
MCP_PROXY_PORT parameter when using non-default port, regardless of
auth status. Also improved code clarity by using params.size check.
---
 client/bin/start.js | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/client/bin/start.js b/client/bin/start.js
index 96556790..e5e32832 100755
--- a/client/bin/start.js
+++ b/client/bin/start.js
@@ -17,19 +17,14 @@ function getClientUrl(port, authDisabled, sessionToken, serverPort) {
   const host = process.env.HOST || "localhost";
   const baseUrl = `http://${host}:${port}`;
 
-  if (authDisabled) {
-    return baseUrl;
-  }
-
   const params = new URLSearchParams();
-  params.set("MCP_PROXY_AUTH_TOKEN", sessionToken);
-
-  // Add server port if it's not the default
   if (serverPort && serverPort !== DEFAULT_MCP_PROXY_LISTEN_PORT) {
     params.set("MCP_PROXY_PORT", serverPort);
   }
-
-  return `${baseUrl}/?${params.toString()}`;
+  if (!authDisabled) {
+    params.set("MCP_PROXY_AUTH_TOKEN", sessionToken);
+  }
+  return params.size > 0 ? `${baseUrl}/?${params.toString()}` : baseUrl;
 }
 
 async function startDevServer(serverOptions) {