-
Notifications
You must be signed in to change notification settings - Fork 492
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 file verifying to UpdateManager #9535
Add file verifying to UpdateManager #9535
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM, is there an easy way to test this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cACK. I made some comments, and I am missing a few tests here.
In the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Max Hillebrand <30683012+MaxHillebrand@users.noreply.github.com>
…tWasabi into addVerifyingToUpdateManager
Resolved smells and refactored the code IRL, is ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK
…Manager Add file verifying to UpdateManager
Instead of #9259
This PR adds different file verifying methods to the UpdateManager with the help of WasabiSignerHelpers, introduced in #9515 .
Once, upon a new release, it verifies the provided
SHA256SUMS.asc
file, with the (also provided)SHA256SUMS.wasabisig
.Then, after the correct installer is downloaded, it verifies the hash of the downloaded file with the expected hash to be found in
SHA256SUMS.asc
.