revault/revault_tx

Exhaustive parsing checks

Aperta

#68 aperta il 27 feb 2021

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)Rust (7 fork)auto 404
enhancementgood first issuehigh prio

Metriche repository

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

Descrizione

Right now, we only sanity-check the PSBTs in from_psbt_serialized() are sane with regard to our internal assumptions (unwrap()s basically). This may be enough, but we may also want complete parsing checks to be sure that whether a transaction is parsed or created with the constructor it will hold to our requirements (eg, no dust amounts, sane feerate, etc..). This is quite some more hairy code though..

Guida contributor

Direzione di ricerca
Implementare controlli di parsing esaustivi in from psbt serialized() per validare i PSBT rispetto a vincoli interni come nessun importo di polvere, fee rate ragionevoli, ecc. Ciò comporta l'aggiunta di logica di validazione per ogni campo del PSBT secondo i requisiti del progetto.
Tech stack
rust
Dominio
backendblockchain
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30