microsoft/vscode-languageserver-node

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

Offen

#1.574 geöffnet am 18.10.2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (397 Forks)auto 404
feature-requesthelp wanted

Repository-Metriken

Stars
 (1.776 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie, wie JSON RPC Parameter in dieser Bibliothek validiert werden, und fügen Sie explizite Prüfungen für Textpositionsfelder hinzu, um sicherzustellen, dass sie innerhalb des von LSP angegebenen Bereichs (0 bis 2^31-1) liegen. Suchen Sie nach vorhandener Validierung in den Protokolltypen und fügen Sie Tests hinzu, um den Überlauffall abzudecken.
Tech Stack
typescriptnodejs
Domain
backendtooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65