Commit 992b4da
committed
find_invalid_x_mcp_header: handle non-str type; bump client conformance timeout
- find_invalid_x_mcp_header: a JSON-Schema array type (e.g. ["string", "null"])
raised TypeError on the frozenset membership check, crashing list_tools() on
modern sessions against untrusted server input. Guard with isinstance(str) so
any non-scalar type cleanly maps to the not-primitive reason. Covers dict/list
type values via two new parametrize cases.
- conformance.yml: bump --timeout to 60s on the client legs. The harness runs
all scenarios via unbounded Promise.all; 40 parallel python processes on a
2-core runner starve sse-retry (the only scenario with a real-time SSE
reconnect wait) past the 30s default.
- everything-server _unseal_state: normalise base64/utf-8 decode errors to the
same INVALID_PARAMS as the HMAC mismatch path.1 parent e418756 commit 992b4da
4 files changed
Lines changed: 14 additions & 2 deletions
File tree
- .github/workflows
- examples/servers/everything-server/mcp_everything_server
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
70 | 73 | | |
71 | 74 | | |
72 | 75 | | |
73 | 76 | | |
| 77 | + | |
74 | 78 | | |
75 | 79 | | |
76 | 80 | | |
77 | 81 | | |
78 | 82 | | |
79 | 83 | | |
| 84 | + | |
80 | 85 | | |
81 | 86 | | |
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
493 | 494 | | |
494 | 495 | | |
495 | 496 | | |
496 | | - | |
| 497 | + | |
| 498 | + | |
| 499 | + | |
| 500 | + | |
497 | 501 | | |
498 | 502 | | |
499 | 503 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
141 | 141 | | |
142 | 142 | | |
143 | 143 | | |
144 | | - | |
| 144 | + | |
| 145 | + | |
145 | 146 | | |
146 | 147 | | |
147 | 148 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
383 | 383 | | |
384 | 384 | | |
385 | 385 | | |
| 386 | + | |
| 387 | + | |
386 | 388 | | |
387 | 389 | | |
388 | 390 | | |
| |||
0 commit comments