From 43d5aaa7980aaa51714175bc9c11d13f39c5d1be Mon Sep 17 00:00:00 2001
From: David East <davideast@users.noreply.github.com>
Date: Tue, 5 Aug 2025 14:44:30 -0400
Subject: fix(mcp): ensure authorization url is valid when containing query
 params (#5545)

Co-authored-by: Jacob Richman <jacob314@gmail.com>
---
 packages/core/src/mcp/oauth-provider.ts | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

(limited to 'packages/core/src/mcp/oauth-provider.ts')

diff --git a/packages/core/src/mcp/oauth-provider.ts b/packages/core/src/mcp/oauth-provider.ts
index 491ec477..5052e8af 100644
--- a/packages/core/src/mcp/oauth-provider.ts
+++ b/packages/core/src/mcp/oauth-provider.ts
@@ -308,7 +308,11 @@ export class MCPOAuthProvider {
       );
     }
 
-    return `${config.authorizationUrl}?${params.toString()}`;
+    const url = new URL(config.authorizationUrl!);
+    params.forEach((value, key) => {
+      url.searchParams.append(key, value);
+    });
+    return url.toString();
   }
 
   /**
-- 
cgit v1.2.3