modelcontextprotocol / modelcontextprotocol/python-sdk
No default timeout for requests (unlike TS SDK)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 24.3k
- Forks
- 4k
- Ø Merge
- 1 T. 1 Std.
- Gemergte PRs (30 T.)
- 31
Beschreibung
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
Spec states: Implementations SHOULD establish timeouts for all sent requests, to prevent hung connections and resource exhaustion.
The TypeScript SDK does have a default timeout of 60 seconds on requests, and the difference between Python & TypeScript is the crux of https://github.com/modelcontextprotocol/typescript-sdk/issues/245 (which also suggests 60 seconds might not be long enough). Also related, TypeScript has a resetTimeoutOnProgress option (defaulting to falst), which would probably be useful if we do introduce a default to the Python SDK.
Example Code
Python & MCP Python SDK
Latest
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Im Issue sind weder Datei noch Test genannt. Beginne daher damit, die Pfade des Python SDK zum Senden von Requests und zur Verarbeitung des Fortschritts zu finden. Vergleiche ihr Verhalten mit dem 60-Sekunden-Standardwert des TypeScript SDK und dessen Diskussion zu resetTimeoutOnProgress. Als abgeschlossen gilt die Implementierung und Verifizierung eines vereinbarten Standard-Timeout-Verhaltens für gesendete Requests.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100