OAuth handler doesn't support redirect URLs with params

Aperta Adatta ai principianti

#2,776 aperta il 3/6/2026 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 2/5
Tempo stimato: 1-3 ore
Idoneità per principianti: 78/100

Tipo di issue: Bug
Chiarezza: Specificata chiaramente
Stato di attività: Attiva
Stack tecnologico: python
Ambito: authentication

Direzione di ricerca

Inizia in src/mcp/client/auth/oauth2.py alle righe 356-357, dove viene assemblato l’URL di autorizzazione prima della chiamata a redirect_handler. Verifica come vengono uniti l’endpoint fornito dal server e i parametri di autorizzazione. Il lavoro è completato quando gli endpoint che contengono già parametri di query producono un URL valido con i parametri esistenti e quelli appena aggiunti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

auth bug P2

Initial Checks

I confirm that I'm using the latest version of MCP Python SDK
I confirm that I searched for my issue in https://github.com/modelcontextprotocol/python-sdk/issues before opening this issue

Description

The oauth redirect handler uses the following logic (https://github.com/modelcontextprotocol/python-sdk/blob/main/src/mcp/client/auth/oauth2.py#L356-L357) to attach parameters to the redirect URL supplied by the server:

        authorization_url = f"{auth_endpoint}?{urlencode(auth_params)}"
        await self.context.redirect_handler(authorization_url)

Unfortunately, this breaks if the auth_endpoint supplied by the server already has URL parameters. For example, at https://api.salesforce.com/.well-known/oauth-authorization-server/platform/mcp/v1/sandbox/platform/sobject-all you can see that the salesforce mcp server has an authorization endpoint of https://test.salesforce.com/services/oauth2/authorize?prompt=select_account.

The end result is that the client tries to visit a URL that loooks like https://test.salesforce.com/services/oauth2/authorize?prompt=select_account?response_type=code&..., which is incorrect.

Example Code

Python & MCP Python SDK

python3 3.13.2
mcp 1.27.2

Lingua principale: Python
Stelle: 24.3k
Fork: 4k
Merge medio: 1g 19m
PR unite (30g): 29

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.