Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move tuf::client::PathTranslator to FileSystemRepository #236

Open
erickt opened this issue Nov 21, 2019 · 1 comment
Open

move tuf::client::PathTranslator to FileSystemRepository #236

erickt opened this issue Nov 21, 2019 · 1 comment

Comments

@erickt
Copy link
Collaborator

erickt commented Nov 21, 2019

tuf::client::PathTranslator allows clients to use an alternative path separator, but according to theupdateframework/specification#63 (comment), the targets metadata path separators should always be /. So it makes more sense that the PathTranslator be a property on tuf::repository::FileSystemRepository, since it's the only thing that really needs to map target paths to files on a file system.

@heartsucker
Copy link
Contributor

At that point it would be easier to just strip it out entirely and then do a split on / and replace it with \\ for windows. No need for a whole trait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants