diff options
| author | Tommaso Sciortino <[email protected]> | 2025-07-29 16:03:39 -0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-07-29 23:03:39 +0000 |
| commit | 327f915610406e4c159495eb734b3ffb0d32ffa4 (patch) | |
| tree | 447b2f3dffb4df2657c7c597f7fc17e4585765eb /packages/core/src | |
| parent | 008051e42d6f4df01c14bfc138e1acaf97cb854a (diff) | |
Fix typo in RFC 9728 impl (#5126)
Diffstat (limited to 'packages/core/src')
| -rw-r--r-- | packages/core/src/mcp/oauth-utils.test.ts | 2 | ||||
| -rw-r--r-- | packages/core/src/mcp/oauth-utils.ts | 4 | ||||
| -rw-r--r-- | packages/core/src/tools/mcp-client.ts | 14 |
3 files changed, 3 insertions, 17 deletions
diff --git a/packages/core/src/mcp/oauth-utils.test.ts b/packages/core/src/mcp/oauth-utils.test.ts index b27d97b3..12871ff2 100644 --- a/packages/core/src/mcp/oauth-utils.test.ts +++ b/packages/core/src/mcp/oauth-utils.test.ts @@ -140,7 +140,7 @@ describe('OAuthUtils', () => { describe('parseWWWAuthenticateHeader', () => { it('should parse resource metadata URI from WWW-Authenticate header', () => { const header = - 'Bearer realm="example", resource_metadata_uri="https://example.com/.well-known/oauth-protected-resource"'; + 'Bearer realm="example", resource_metadata="https://example.com/.well-known/oauth-protected-resource"'; const result = OAuthUtils.parseWWWAuthenticateHeader(header); expect(result).toBe( 'https://example.com/.well-known/oauth-protected-resource', diff --git a/packages/core/src/mcp/oauth-utils.ts b/packages/core/src/mcp/oauth-utils.ts index 6dad17c8..64fd68be 100644 --- a/packages/core/src/mcp/oauth-utils.ts +++ b/packages/core/src/mcp/oauth-utils.ts @@ -198,8 +198,8 @@ export class OAuthUtils { * @returns The resource metadata URI if found */ static parseWWWAuthenticateHeader(header: string): string | null { - // Parse Bearer realm and resource_metadata_uri - const match = header.match(/resource_metadata_uri="([^"]+)"/); + // Parse Bearer realm and resource_metadata + const match = header.match(/resource_metadata="([^"]+)"/); if (match) { return match[1]; } diff --git a/packages/core/src/tools/mcp-client.ts b/packages/core/src/tools/mcp-client.ts index d175af1f..b87b2124 100644 --- a/packages/core/src/tools/mcp-client.ts +++ b/packages/core/src/tools/mcp-client.ts @@ -146,20 +146,6 @@ export function getMCPDiscoveryState(): MCPDiscoveryState { } /** - * Parse www-authenticate header to extract OAuth metadata URI. - * - * @param wwwAuthenticate The www-authenticate header value - * @returns The resource metadata URI if found, null otherwise - */ -function _parseWWWAuthenticate(wwwAuthenticate: string): string | null { - // Parse header like: Bearer realm="MCP Server", resource_metadata_uri="https://..." - const resourceMetadataMatch = wwwAuthenticate.match( - /resource_metadata_uri="([^"]+)"/, - ); - return resourceMetadataMatch ? resourceMetadataMatch[1] : null; -} - -/** * Extract WWW-Authenticate header from error message string. * This is a more robust approach than regex matching. * |
