diff options
| author | David East <[email protected]> | 2025-08-05 14:44:30 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-08-05 18:44:30 +0000 |
| commit | 43d5aaa7980aaa51714175bc9c11d13f39c5d1be (patch) | |
| tree | 5986842dbf79dcf80db350a548ccae73fb2b04a8 /packages/core/src/mcp/oauth-provider.ts | |
| parent | 5c8268b6f44e96ef1975999baac71c022875c321 (diff) | |
fix(mcp): ensure authorization url is valid when containing query params (#5545)
Co-authored-by: Jacob Richman <[email protected]>
Diffstat (limited to 'packages/core/src/mcp/oauth-provider.ts')
| -rw-r--r-- | packages/core/src/mcp/oauth-provider.ts | 6 |
1 files changed, 5 insertions, 1 deletions
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(); } /** |