revault/revault_tx

Exhaustive parsing checks

Ouverte

#68 ouverte le 27 févr. 2021

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Rust (7 forks)auto 404
enhancementgood first issuehigh prio

Métriques du dépôt

Stars
 (15 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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..

Guide contributeur

Direction de recherche
Implémenter des contrôles d'analyse exhaustifs dans from psbt serialized() pour valider les PSBT par rapport aux contraintes internes telles que pas de montants de poussière, taux de frais raisonnables, etc. Cela implique d'ajouter une logique de validation pour chaque champ du PSBT selon les exigences du projet.
Stack technique
rust
Domaine
backendblockchain
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30