microsoft/vscode-languageserver-node

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

Aberta

#1.574 aberto em 18 de out. de 2024

Ver no GitHub
 (4 comentários) (0 reação) (0 responsável)TypeScript (397 forks)auto 404
feature-requesthelp wanted

Métricas do repositório

Stars
 (1.776 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Investigue como os parâmetros JSON RPC são validados nesta biblioteca e adicione verificações explícitas para os campos de posição de texto para garantir que estejam dentro do intervalo especificado pelo LSP (0 a 2^31-1). Procure por validação existente nos tipos de protocolo e adicione testes para cobrir o caso de estouro.
Pilha de tecnologia
typescriptnodejs
Domain
backendtooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65