Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Reject `subscriptions/listen` without `notifications`

Ouverte Adaptée aux débutants
#1,196 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
68/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
Active
Stack technique
java

Piste de recherche

Commencez par localiser SubscriptionsFeature et suivre la façon dont il analyse les paramètres de subscriptions/listen, puis consultez les tests à proximité pour repérer les requêtes d’abonnement mal formées. Ajoutez des tests vérifiant que l’absence de notifications et sa valeur null sont rejetées, tandis que {} reste valide, et confirmez que la réponse est -32602 (et HTTP 400 le cas échéant).

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

area/server bug P2

Spec:

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.

Langage dominant
Java
Étoiles
3.7k
Forks
1.2k
Merge moyen
16 h 20 min
PR mergées (30 j)
18

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de modelcontextprotocol/java-sdk

Toutes les issues de modelcontextprotocol/java-sdk

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.