modelcontextprotocol / modelcontextprotocol/java-sdk
Reject listRoots if not supported by client, without sending any request
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 3.7k
- Forks
- 1.1k
- Merge moyen
- 1 j 15 h
- PR mergées (30 j)
- 9
Description
Some requests in McpAsyncServerExchange make a point of failing fast if functionality not supported, such the condition on line 146
listRoots does not do the same
For example, add this
if (this.clientCapabilities.roots() == null) {
return Mono.error(new IllegalStateException("Client must be configured with root listing capabilities"));
}
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Lisez mcp-core/src/main/java/io/modelcontextprotocol/server/McpAsyncServerExchange.java, en particulier la vérification de capability existante autour des lignes 141-151 et le point d’entrée listRoots autour des lignes 227-230. Confirmez qu’une capability roots non prise en charge est rejetée avant l’envoi de toute requête, avec le comportement d’erreur spécifié, puis exécutez les tests pertinents du projet.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- api
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 82/100