Reject `subscriptions/listen` without `notifications`
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 68/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- java
- Bereich
- backend-api-design
Rechercherichtung
Beginne damit, SubscriptionsFeature zu finden und nachzuverfolgen, wie es die Parameter von subscriptions/listen parst. Prüfe anschließend die Tests in der Nähe auf fehlerhafte Abonnementanfragen. Ergänze Tests dafür, dass fehlende und nullwertige notifications abgelehnt werden, während {} gültig bleibt, und bestätige, dass die Antwort -32602 lautet (und gegebenenfalls HTTP 400).
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Spec:
- Schema:
SubscriptionsListenRequestParams(notifications: SubscriptionFilteris a required field) - Subscriptions: Opening a Stream
Current behavior
If notifications is missing or null, SubscriptionsFeature treats it as an empty filter, acknowledges the subscription and keeps an idle stream open.
Proposal
Answer with -32602 Invalid params (400 on HTTP), as for other malformed params. An explicit empty object {} stays valid.
- Vorherrschende Sprache
- Java
- Sterne
- 3.7k
- Forks
- 1.2k
- Ø Merge
- 13 Std. 30 Min.
- Gemergte PRs (30 T.)
- 23
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus modelcontextprotocol/java-sdk
-
HttpServletStreamableServerTransportProvider: GET stream sends no status or headers until the first eventEvtl. vergeben @karthiksenv hat das vor 19 Tagen übernommen. Offenarea/server area/transport P3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
modelcontextprotocol/java-sdk#1155 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Client request handlers that complete empty send no JSON-RPC responseEvtl. vergeben @1fanwang hat das vor 48 Tagen übernommen. Offenarea/client bug P2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
modelcontextprotocol/java-sdk#1124 · 5 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
ServerCapabilities.logging is added unconditionally, overriding the caller's explicit capabilitiesEvtl. vergeben @1yuxiangJ hat das vor 63 Tagen übernommen. Offenbug P2 ready for work
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
modelcontextprotocol/java-sdk#1086 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Reject listRoots if not supported by client, without sending any requestEvtl. vergeben @nikita-kibitkin hat das vor 87 Tagen übernommen. Offenenhancement good first issue P3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
modelcontextprotocol/java-sdk#1067 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
StdioClientTransport missing explicit UTF-8 charset in InputStreamReader (same issue as #295, but on client side)Evtl. vergeben @suryateja-g13 hat das vor 158 Tagen übernommen. Offenbug P2 ready for work
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
modelcontextprotocol/java-sdk#898 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in modelcontextprotocol/java-sdk
Ähnliche Issues
-
human-required sync-failed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Azure/osdu-spi-file#63 ·
Maintainer antworten meist innerhalb von 1 Tag
-
frontend quick win
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
CodeForPhilly/benefit-decision-toolkit#571 ·
Maintainer antworten meist innerhalb von 1 Tag
-
approved
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
checkstyle/checkstyle#22129 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Add Hardcharger JDBC driverEvtl. vergeben @prmoore77 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag