Metadata API: SuccinctRoles.is_delegated_role() accepts non-canonical bin names
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 42/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 1.7k
- Forks
- 307
- Ø Merge
- 8 Std. 11 Min.
- Gemergte PRs (30 T.)
- 12
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus theupdateframework/python-tuf
-
switch to main branch?Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
theupdateframework/python-tuf#3001 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100
theupdateframework/python-tuf#2979 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Use Immutable ReleasesOffenenhancement github_actions
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
theupdateframework/python-tuf#2920 · 1 Kommentar · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
Investigate/test fetcher retriesEvtl. vergeben @ishaanxgupta hat das vor 291 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
theupdateframework/python-tuf#2842 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
ngclient: Be better with concurrent instancesEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
theupdateframework/python-tuf#2836 · 7 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in theupdateframework/python-tuf
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
conda-forge/conda-forge-bot#7025 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:ci
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 74/100
catincloud-labs/dagster-dex#149 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
home-assistant/core#188424 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Review backlog (docs-currency): README 'CI run' link lands on a run whose overall badge is cancelledOffenagent/reviewer needs-kind needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 63/100
projectbluefin/utah#880 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
LSan reports obj_gen leak on post-parse usage() exits (aarch64)Evtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
redis/memtier_benchmark#850 ·
Maintainer antworten meist innerhalb von 1 Tag