-
Notifications
You must be signed in to change notification settings - Fork 438
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
Add MD5 for each box. #123
Comments
Unfortunately, we rely on people submitting data to us - many have added MD5 sums, but most have not. We're looking to improve things in the way we collate this data in the near-term (See #29 for details), so perhaps incorporating the MD5 sum on that would be good, but it's not something I can see anyone pushing for. Given most people do vagrant box add {boxname} {boxurl} and that doesn't show an MD5 or SHA1, is this still relevant? |
You're totally right, I did see that this issue exists later on (got to #221 by a Google search).
Not at all, only if authors wish to sign their boxes. People can choose to only use signed boxes if they wish, and it will introduce reputation and trust into the system. As for md5/sha1 digest vs digital signature well, something is better than nothing. It's true that doing a |
I should add, we do actually have a single box (a Fedora image) which lists both it's SHA1 and MD5 sum... perhaps we could encourage that to be the template? |
@JonTheNiceGuy I'd actually could hash everything we have, and even remove those images, which links are unavailable anymore. But I see that you had few PR's about new template. I'd also vote for new template first. When you'll have new template ready, I'd do cleanup for you. 👍 |
It would help to verity if the downloaded file is correct.
The text was updated successfully, but these errors were encountered: