add_key() and string measurements don't check Lente column sizes
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 76/100
Direzione di ricerca
Inizia leggendo docs/source/_traceability.rst per conoscere i limiti di colonna documentati, quindi individua ResultBaseKeysV1.add_key() e measurement() negli script. Verifica come vengono registrati gli errori esistenti e come viene restituito RECORD_RESULT_INTERNAL_ERROR. Il lavoro è completato quando le chiavi e i valori stringa troppo lunghi vengono rilevati sulla stazione prima che Lente inserisca il record.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
ResultBaseKeysV1.add_key() stores "<name>:<value>" with no length check, but Lente's record.key0-key4 columns are String(64). measurement() checks the name (128) but not a string value, and measurement.value is String(128).
A longer key or string value passes on Prism and only fails when Lente inserts the record, so the result goes to quarantine.
Suggested fix: check the length in add_key() (return False and log) and in measurement() for str values (RECORD_RESULT_INTERNAL_ERROR), so the problem shows on the station during development. The limits are now documented in docs/source/_traceability.rst.
- Lingua principale
- C++
- Stelle
- 4
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 sistemicorp/scripts
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
sistemicorp/scripts#60 ·
-
bug documentation
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
sistemicorp/scripts#59 ·
-
Add P1150 Hardware DriverForse di nuovo libera @sistemicorp l’ha presa 187 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
sistemicorp/scripts#52 · 1 assegnatario ·
-
[BOND] Changes for rev3Forse di nuovo libera @sistemicorp l’ha presa 375 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
sistemicorp/scripts#26 · 1 assegnatario ·
Tutte le issue di sistemicorp/scripts
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
bug code-review testing tooling
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
aristocratos/btop#1945 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
boykopovar/AnyPS5#11291 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
k2-fsa/sherpa-onnx#4100 ·
I maintainer di solito rispondono entro 2 giorni