Metadata API: SuccinctRoles.is_delegated_role() accepts non-canonical bin names
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 42/100
Piste de recherche
Read SuccinctRoles.is_delegated_role() and the tests for delegated role names, then check how get_roles() and get_role_for_target() generate canonical names. Add coverage for the non-canonical suffixes listed in the issue and confirm they are rejected while generated names remain accepted. The issue author says a fix and tests are ready, so check the thread for their promised pull request before starting.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
SuccinctRoles.is_delegated_role() validates the bin suffix with int(suffix, 16). int() also accepts a leading sign, surrounding whitespace, underscores between digits and upper case digits, so with SuccinctRoles([], 1, 5, "bin") the names bin-+f, bin- f, bin-f , bin-0F are reported as delegated (and bin-1_0 with bit_length 12), although get_roles()/get_role_for_target() only ever produce zero-padded lower case names like bin-0f. Targets.get_delegated_role() relies on this check. I'd expect only the exact generated names to be accepted.
I have a fix with tests ready and will open a PR.
- Langage dominant
- Python
- Étoiles
- 1.7k
- Forks
- 307
- Merge moyen
- 8 h 11 min
- PR mergées (30 j)
- 12
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de theupdateframework/python-tuf
-
switch to main branch?Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
theupdateframework/python-tuf#3001 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
theupdateframework/python-tuf#2979 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Use Immutable ReleasesOuverteenhancement github_actions
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
theupdateframework/python-tuf#2920 · 1 commentaire · 2 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
Investigate/test fetcher retriesPeut-être pris @ishaanxgupta l’a pris il y a 291 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
theupdateframework/python-tuf#2842 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
ngclient: Be better with concurrent instancesPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
theupdateframework/python-tuf#2836 · 7 commentaires ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de theupdateframework/python-tuf
Issues similaires
-
The changelog workflow excludes otelbot but not dependabot, so every Dependabot pull request fails the changelog checkPeut-être pris @ocelotl l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
open-telemetry/opentelemetry-python#5825 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
cuga-project/cuga-agent#1020 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 70/100
microsoft/playwright#43640 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
stac-utils/stac-fastapi-elasticsearch-opensearch#965 ·
Les mainteneurs répondent en général sous 2 jours
-
from-review-extraction library privacy python severity:suggested
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
LearningCircuit/local-deep-research#7951 ·
Les mainteneurs répondent en général sous 2 jours