fix: don't treat non-success HTTP codes as transport errors

alexhancock · alexhancock · commit e90f3ef12535 · 2026-01-14T13:17:37.000-05:00
diff --git a/crates/rmcp/src/transport/common/reqwest/streamable_http_client.rs b/crates/rmcp/src/transport/common/reqwest/streamable_http_client.rs
@@ -121,7 +121,6 @@ impl StreamableHttpClient for reqwest::Client {
             }
         }
         let status = response.status();
-        let response = response.error_for_status()?;
         if matches!(
             status,
             reqwest::StatusCode::ACCEPTED | reqwest::StatusCode::NO_CONTENT

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,6 @@ impl StreamableHttpClient for reqwest::Client {`
`121`	`121`	`}`
`122`	`122`	`}`
`123`	`123`	`let status = response.status();`
`124`		`- let response = response.error_for_status()?;`
`125`	`124`	`if matches!(`
`126`	`125`	`status,`
`127`	`126`	`reqwest::StatusCode::ACCEPTED \| reqwest::StatusCode::NO_CONTENT`