trezor/definitions

Check and clip threshold values.

Ouverte

#85 ouverte le 12 août 2026

Voir sur GitHub
 (0 commentaire) (1 réaction) (1 personne assignée)Python (7 forks)github user discovery
enhancementerc7730ethereumgood first issue

Métriques du dépôt

Stars
 (3 étoiles)
Métriques de merge PR
 (Merge moyen 5j 11h) (8 PRs mergées en 30 j)

Description

The spec says the values should be ABOVE the threshold, but practically it is above OR EQUAL. This might be updated as well: https://github.com/ethereum/clear-signing-erc7730-registry/issues/2863

We can keep a track of it, but until then, we could consider clipping and logging a field's max value to MAX_VALUE - 1 in the definition itself to avoid errors.

It is not a priority since this merged: https://github.com/ethereum/clear-signing-erc7730-registry/pull/2870 but would be good to have.

OR

fixed in Firmware.

Guide contributeur

Direction de recherche
Examinez la logique de validation des seuils dans le dépôt definitions, identifiez où l'égalité est autorisée au lieu d'une stricte supériorité, et envisagez de limiter la valeur maximale du champ à MAX VALUE 1 ou de l'aligner sur le comportement du firmware.
Stack technique
python
Domaine
blockchainbackend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
Plutôt claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60