node-red-node-cbor cannot handle BigInt
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend
Direzione di ricerca
Inizia in parsers/cbor/70-cbor.js, intorno al codice di serializzazione collegato, e riproduci la decodifica di un timestamp o di un altro valore BigInt. Verifica che i buffer validi vengano decodificati senza l'errore JSON e che gli errori riportino l'errore sottostante; esamina inoltre l'aggiornamento richiesto di cbor-x. Nell'issue non è indicato alcun file di test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Which node are you reporting and issue on?
node-red-node-cbor
What are the steps to reproduce?
send a timestamp as integer to decode - the decoder exits with the error message "Bad Decode"
What happens?
the node-red-node-cbor plugin serializes the decoded result with JSON: https://github.com/node-red/node-red-nodes/blob/06a9c48395ab8ba707a0f265162df5c0c81dd74b/parsers/cbor/70-cbor.js#L19
As JSON does not support BigInt, this creates an error message when you are decoding a timestamp for example.
What did you expect to happen?
- The decoder should decode correctly sent buffers without error message.
- The error message should correspond to the actual error.
Proposed solution
- I propose to add a replacer function and convert BigInt and other unsupported formats.
- The error message "not a cbor buffer" is not always correct. I would prefer to use the actual error message.
Additionally, cbor-x library is out of date and should be updated.
- Lingua principale
- JavaScript
- Stelle
- 1.1k
- Fork
- 610
- Merge medio
- 18h 41m
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di node-red/node-red-nodes
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 58/100
node-red/node-red-nodes#1142 · 11 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
node-red/node-red-nodes#1141 · 15 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
node-red/node-red-nodes#1130 · 4 commenti ·
-
dependencies
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
node-red/node-red-nodes#1121 · 5 commenti ·
-
upstream
Difficoltà 3/5 1-2 giorni Idoneità per principianti 50/100
node-red/node-red-nodes#1116 · 5 commenti ·
Tutte le issue di node-red/node-red-nodes
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
capricorn86/happy-dom#2485 ·
I maintainer di solito rispondono entro 2 giorni
-
area:space-accuracy good first issue track:data
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Sara-Managed-Projects/space-radar#904 ·
I maintainer di solito rispondono entro 1 giorno