From 41a240a3893903afc58539cdaa5d14f4c7f70f3d Mon Sep 17 00:00:00 2001 From: hongkunyoo Date: Wed, 21 May 2025 21:41:28 +0900 Subject: [PATCH] Remove `lstrip` in each oauth endpoint `lstrip` strips left `/` which results in `ISSUER_URLauthorize` --- src/mcp/server/auth/routes.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mcp/server/auth/routes.py b/src/mcp/server/auth/routes.py index 4c56ca247..19d72ac5e 100644 --- a/src/mcp/server/auth/routes.py +++ b/src/mcp/server/auth/routes.py @@ -167,10 +167,10 @@ def build_metadata( revocation_options: RevocationOptions, ) -> OAuthMetadata: authorization_url = modify_url_path( - issuer_url, lambda path: path.rstrip("/") + AUTHORIZATION_PATH.lstrip("/") + issuer_url, lambda path: path.rstrip("/") + AUTHORIZATION_PATH ) token_url = modify_url_path( - issuer_url, lambda path: path.rstrip("/") + TOKEN_PATH.lstrip("/") + issuer_url, lambda path: path.rstrip("/") + TOKEN_PATH ) # Create metadata metadata = OAuthMetadata( @@ -194,13 +194,13 @@ def build_metadata( # Add registration endpoint if supported if client_registration_options.enabled: metadata.registration_endpoint = modify_url_path( - issuer_url, lambda path: path.rstrip("/") + REGISTRATION_PATH.lstrip("/") + issuer_url, lambda path: path.rstrip("/") + REGISTRATION_PATH ) # Add revocation endpoint if supported if revocation_options.enabled: metadata.revocation_endpoint = modify_url_path( - issuer_url, lambda path: path.rstrip("/") + REVOCATION_PATH.lstrip("/") + issuer_url, lambda path: path.rstrip("/") + REVOCATION_PATH ) metadata.revocation_endpoint_auth_methods_supported = ["client_secret_post"]