-
Notifications
You must be signed in to change notification settings - Fork 280
Closed
Labels
backlogIssues to address with priority for current development goalsIssues to address with priority for current development goalsdiscussionDiscussions related to the design, implementation and operation of the projectDiscussions related to the design, implementation and operation of the project
Description
Description of issue or feature request:
Delegation role names are not restricted in any way in the spec, but they are targets metadata role names.
They could be "."
, "../../filename"
or 1.role
.
The problem is that at some point those delegation role names are used when constructing an URL used
to download the delegated target metadata file:
https://github.com/theupdateframework/tuf/blob/e9106b59cdb5bbfb4260c5ffc3144e79f8f9596a/tuf/ngclient/updater.py#L287 which is likely to be a problem.
Current behavior:
No validation is used for Delegation role names.
Expected behavior:
Escape special symbols like .
or \
.
Metadata
Metadata
Assignees
Labels
backlogIssues to address with priority for current development goalsIssues to address with priority for current development goalsdiscussionDiscussions related to the design, implementation and operation of the projectDiscussions related to the design, implementation and operation of the project