microsoft/vscode-languageserver-node

Overflow value(exceeding uinteger) passed into jsonrpc but still work

Ouverte

#1 574 ouverte le 18 oct. 2024

Voir sur GitHub
 (4 commentaires) (0 réaction) (0 personne assignée)TypeScript (397 forks)auto 404
feature-requesthelp wanted

Métriques du dépôt

Stars
 (1 776 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

The reason is that I found that the fmt feature of my lsp server cannot be used in IntelliJ, but it still work in vscode. After investigation, I found that I assigned a u32:Max to formatResponse.textedit.range.line in the format's Response I implemented the lsp server in rust, and u32 was used as the range type in rust's lsp-type, but the range of this value in lsp should be 0-2^31 -1. I think this is the cause of the problem

But I'm not sure whether vscode has special handling for this situation or just forgot to check

https://github.com/redhat-developer/lsp4ij/issues/579 https://github.com/kcl-lang/kcl/pull/1698

Guide contributeur

Direction de recherche
Enquêtez sur la façon dont les paramètres JSON RPC sont validés dans cette bibliothèque, et ajoutez des vérifications explicites pour les champs de position de texte afin de garantir qu'ils se trouvent dans la plage spécifiée par LSP (0 à 2^31-1). Recherchez la validation existante dans les types de protocole et ajoutez des tests pour couvrir le cas de dépassement.
Stack technique
typescriptnodejs
Domaine
backendtooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65