microsoft/vscode-languageserver-node

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

Aperta

#1574 aperta il 18 ott 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)TypeScript (397 fork)auto 404
feature-requesthelp wanted

Metriche repository

Star
 (1776 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga su come i parametri JSON RPC vengono validati in questa libreria e aggiungi controlli espliciti per i campi di posizione del testo per assicurarti che siano entro l'intervallo specificato da LSP (0-2^31-1). Cerca la validazione esistente nei tipi di protocollo e aggiungi test per coprire il caso di overflow.
Tech stack
typescriptnodejs
Dominio
backendtooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65