Merge pull request #31 from Beyond-Better/staging

fix well-known protocol

Author: cngarrison

deno.jsonc

@@ -1,6 +1,6 @@
 {
   "name": "@beyondbetter/bb-mcp-server",
-  "version": "0.1.18",
+  "version": "0.1.19",
   "description": "Comprehensive library for building Deno-based MCP servers",
   "license": "MIT",
   "copyright": "2025 - Beyond Better <charlie@beyondbetter.app>",
@@ -40,18 +40,18 @@
     "cleanup:orphaned-chunks:verbose": "deno run --allow-read --allow-write --allow-env scripts/cleanup-orphaned-chunks.ts --verbose"
   },
   "imports": {
-    "mcp": "npm:@modelcontextprotocol/sdk@1.18.2",
-    "mcp/client/": "npm:@modelcontextprotocol/sdk@1.18.2/client/",
-    "mcp/client/index.js": "npm:@modelcontextprotocol/sdk@1.18.2/client/index.js",
-    "mcp/client/auth.js": "npm:@modelcontextprotocol/sdk@1.18.2/client/auth.js",
-    "mcp/client/stdio.js": "npm:@modelcontextprotocol/sdk@1.18.2/client/stdio.js",
-    "mcp/client/streamableHttp.js": "npm:@modelcontextprotocol/sdk@1.18.2/client/streamableHttp.js",
-    "mcp/server/index.js": "npm:@modelcontextprotocol/sdk@1.18.2/server/index.js",
-    "mcp/server/stdio.js": "npm:@modelcontextprotocol/sdk@1.18.2/server/stdio.js",
-    "mcp/server/streamableHttp.js": "npm:@modelcontextprotocol/sdk@1.18.2/server/streamableHttp.js",
-    "mcp/shared/transport.js": "npm:@modelcontextprotocol/sdk@1.18.2/shared/transport.js",
-    "mcp/shared/auth.js": "npm:@modelcontextprotocol/sdk@1.18.2/shared/auth.js",
-    "mcp/types.js": "npm:@modelcontextprotocol/sdk@1.18.2/types.js",
+    "mcp": "npm:@modelcontextprotocol/sdk@1.25.3",
+    "mcp/client/": "npm:@modelcontextprotocol/sdk@1.25.3/client/",
+    "mcp/client/index.js": "npm:@modelcontextprotocol/sdk@1.25.3/client/index.js",
+    "mcp/client/auth.js": "npm:@modelcontextprotocol/sdk@1.25.3/client/auth.js",
+    "mcp/client/stdio.js": "npm:@modelcontextprotocol/sdk@1.25.3/client/stdio.js",
+    "mcp/client/streamableHttp.js": "npm:@modelcontextprotocol/sdk@1.25.3/client/streamableHttp.js",
+    "mcp/server/index.js": "npm:@modelcontextprotocol/sdk@1.25.3/server/index.js",
+    "mcp/server/stdio.js": "npm:@modelcontextprotocol/sdk@1.25.3/server/stdio.js",
+    "mcp/server/streamableHttp.js": "npm:@modelcontextprotocol/sdk@1.25.3/server/streamableHttp.js",
+    "mcp/shared/transport.js": "npm:@modelcontextprotocol/sdk@1.25.3/shared/transport.js",
+    "mcp/shared/auth.js": "npm:@modelcontextprotocol/sdk@1.25.3/shared/auth.js",
+    "mcp/types.js": "npm:@modelcontextprotocol/sdk@1.25.3/types.js",
     "json-schema": "https://esm.sh/v135/@types/json-schema@7.0.15/index.d.ts",
     "zod": "npm:zod@^3.22.4",
     "zod-to-json-schema": "npm:zod-to-json-schema@^3.24.6",

src/lib/utils/UrlUtils.ts

@@ -24,13 +24,15 @@ export function reconstructOriginalUrl(request: Request): URL {
   // Parse the current URL to get path and query parameters
   const currentUrl = new URL(request.url);
 
+  //console.log('reconstructOriginalUrl', { forwardedProto, currentUrlProto: currentUrl.protocol});
   // Reconstruct with proper protocol and host
-  const protocol = `${forwardedProto}:` || currentUrl.protocol;
+  const protocol = forwardedProto ? `${forwardedProto}:` : currentUrl.protocol;
   const host = forwardedHost || currentUrl.host;
 
   const reconstructedUrl = new URL(
     `${protocol}//${host}${currentUrl.pathname}${currentUrl.search}${currentUrl.hash}`,
   );
+  //console.log('reconstructOriginalUrl', { reconstructedUrl});
 
   return reconstructedUrl;
 }