Commit 87e40ac
Deprecate the SSE transport
HTTP+SSE was superseded by Streamable HTTP in protocol revision
2025-03-26, and the spec now documents it only for backwards
compatibility. Emit a DeprecationWarning from both SSE transport
primitives — `sse_client` (client) and `SseServerTransport` (server) —
and note the deprecation in their docstrings. The high-level
`MCPServer.sse_app()` / `run(transport="sse")` build on
`SseServerTransport`, so they surface the warning too.
Unlike the WebSocket deprecation, SSE is used internally (the high-level
server app, `ClientSessionGroup`, the `python -m mcp.client` CLI) and
across the transport-parametrized interaction test suite. A
`typing_extensions.deprecated` decorator would therefore force a
`# pyright: ignore[reportDeprecated]` at every internal call site, which
AGENTS.md discourages. A plain `warnings.warn` gives the same runtime
signal with no static-analysis fallout.
`filterwarnings = ["error"]` would otherwise promote the SDK's own
deprecation notice to a test error wherever SSE is exercised, so add two
scoped `ignore` entries (mirroring the existing pywin32 one) and assert
the warnings are emitted in tests/shared/test_sse.py.
Closes #2278
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>1 parent cf110e3 commit 87e40ac
5 files changed
Lines changed: 52 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
149 | 149 | | |
150 | 150 | | |
151 | 151 | | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
152 | 156 | | |
153 | 157 | | |
154 | 158 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
210 | 210 | | |
211 | 211 | | |
212 | 212 | | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
213 | 219 | | |
214 | 220 | | |
215 | 221 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
| 2 | + | |
2 | 3 | | |
3 | 4 | | |
4 | 5 | | |
| |||
39 | 40 | | |
40 | 41 | | |
41 | 42 | | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
42 | 47 | | |
43 | 48 | | |
44 | 49 | | |
| |||
51 | 56 | | |
52 | 57 | | |
53 | 58 | | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
54 | 66 | | |
55 | 67 | | |
56 | 68 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
| 40 | + | |
40 | 41 | | |
41 | 42 | | |
42 | 43 | | |
| |||
70 | 71 | | |
71 | 72 | | |
72 | 73 | | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
73 | 78 | | |
74 | 79 | | |
75 | 80 | | |
| |||
100 | 105 | | |
101 | 106 | | |
102 | 107 | | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
103 | 115 | | |
104 | 116 | | |
105 | 117 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
108 | 108 | | |
109 | 109 | | |
110 | 110 | | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
111 | 129 | | |
112 | 130 | | |
113 | 131 | | |
| |||
0 commit comments