modelcontextprotocol/python-sdk
Watch
1
0
Fork
You've already forked python-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: avoid double JSON parsing (#1917)

Browse source
This commit is contained in:
Marcelo Trylesinski 2026-01-21 13:07:05 +01:00 • committed by GitHub
commit ae7793efd6
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/mcp/server/auth/handlers/register.py
View file

@ -32,10 +32,8 @@ class RegistrationHandler:
async def handle(self, request: Request) -> Response: async def handle(self, request: Request) -> Response:
# Implements dynamic client registration as defined in https://datatracker.ietf.org/doc/html/rfc7591#section-3.1 # Implements dynamic client registration as defined in https://datatracker.ietf.org/doc/html/rfc7591#section-3.1
try: try:
# Parse request body as JSON body = await request.body()
# TODO(Marcelo): This is unnecessary. We should use `request.body()`. client_metadata = OAuthClientMetadata.model_validate_json(body)
body = await request.json()
client_metadata = OAuthClientMetadata.model_validate(body)
# Scope validation is handled below # Scope validation is handled below
except ValidationError as validation_error: except ValidationError as validation_error:

4
src/mcp/server/fastmcp/server.py
View file

@ -114,7 +114,7 @@ class FastMCP(Generic[LifespanResultT]):
website_url: str | None = None, website_url: str | None = None,
icons: list[Icon] | None = None, icons: list[Icon] | None = None,
version: str | None = None, version: str | None = None,
auth_server_provider: (OAuthAuthorizationServerProvider[Any, Any, Any] | None) = None, auth_server_provider: OAuthAuthorizationServerProvider[Any, Any, Any] | None = None,
token_verifier: TokenVerifier | None = None, token_verifier: TokenVerifier | None = None,
*, *,
tools: list[Tool] | None = None, tools: list[Tool] | None = None,
@ -123,7 +123,7 @@ class FastMCP(Generic[LifespanResultT]):
warn_on_duplicate_resources: bool = True, warn_on_duplicate_resources: bool = True,
warn_on_duplicate_tools: bool = True, warn_on_duplicate_tools: bool = True,
warn_on_duplicate_prompts: bool = True, warn_on_duplicate_prompts: bool = True,
lifespan: (Callable[[FastMCP[LifespanResultT]], AbstractAsyncContextManager[LifespanResultT]] | None) = None, lifespan: Callable[[FastMCP[LifespanResultT]], AbstractAsyncContextManager[LifespanResultT]] | None = None,
auth: AuthSettings | None = None, auth: AuthSettings | None = None,
): ):
self.settings = Settings( self.settings = Settings(