Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Streamable HTTP client: a 401 or 403 with a JSON-RPC error body and no WWW-Authenticate loses its HTTP status

Aperta Adatta ai principianti
  • Qualcosa non va?
#1,339 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 3 giorni

@aton-of-data ci sta già lavorando.

Dal 11/10/2026.

  • #1341 di @aton-of-data — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
authentication, backend

Direzione di ricerca

Leggi crates/rmcp/src/transport/common/reqwest/streamable_http_client.rs, in particolare il ramo delle risposte non riuscite intorno alle righe 276–306 e i controlli precedenti per 401/403. Confronta la gestione con legacy_discover_response e il percorso di errore in testo semplice. Il lavoro è completato quando una risposta 401 o 403 con un corpo di errore JSON-RPC conserva lo stato HTTP per chi effettua le chiamate; esegui i test pertinenti del client HTTP Streamable.

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

Descrizione

bug P2 ready for work T-security T-transport

When a Streamable HTTP server answers a POST with 401 Unauthorized (or 403 Forbidden) without a WWW-Authenticate header and with a JSON-RPC error body, the reqwest client returns Ok(StreamableHttpPostResponse::Json(..)). The caller then gets an ordinary JSON-RPC error, ClientInitializeError::JsonRpcError during initialize or ServiceError::McpError after it, and cannot tell that the server rejected the credentials.

Where

The non-success branch parses the body with parse_json_rpc_error before it looks at the status:

https://github.com/modelcontextprotocol/rust-sdk/blob/c1a8eea312613519b93f53b89a1dc74dc993f5b0/crates/rmcp/src/transport/common/reqwest/streamable_http_client.rs#L276-L306

AuthRequired and InsufficientScope come only from the checks above it, which require WWW-Authenticate. The same 401 with a plain-text body becomes UnexpectedServerResponse("HTTP 401 Unauthorized: …"), so the status survives on that path but not on the JSON one.

Reproduce

rmcp 3.5.1 (main has the same code). The server answers initialize with:

HTTP/1.1 401 Unauthorized
Content-Type: application/json

{"jsonrpc":"2.0","id":0,"error":{"code":-32001,"message":"Unauthorized"}}

serve_client over StreamableHttpClientTransport::with_client(reqwest::Client, config) fails with ClientInitializeError::JsonRpcError (code -32001, message "Unauthorized"). Nothing in the error says HTTP 401.

Why it matters

A client that sends a static token through custom_headers wants to tell the user that the token was rejected. An OAuth client wants to start or refresh authorization. Both need the status. The MCP authorization spec asks servers that implement authorization to send WWW-Authenticate with a 401, but a server that only takes a static token is not bound by that and may answer with a JSON-RPC error body instead.

Possible fixes
  • Handle 401 and 403 before the JSON-RPC branch, as legacy_discover_response already does ("Keep authentication failures and server errors on their original paths"). For example, return AuthRequired or InsufficientScope for every 401 or 403, with the challenge optional, or return UnexpectedServerResponse("HTTP 401 …") as the plain-text path does.
  • Or keep the JSON-RPC error and carry the HTTP status with it.
Lingua principale
Rust
Stelle
4k
Fork
662
Merge medio
3g 10h
PR unite (30g)
33

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

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

Altre issue di modelcontextprotocol/rust-sdk

Tutte le issue di modelcontextprotocol/rust-sdk

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.