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

Verify all checksums in prepare-system.sh #1642

Merged
merged 3 commits into from
Aug 28, 2018

Conversation

chirhonul
Copy link
Contributor

Also improve style in general.

We do this for the jce_policy-8.zip file, but we also fetch the BC
.jar over http and it could be trivially MitM'ed by a network adversary
before this patch.
Consistently indent with four spaces.

Use ${foo} instead of $foo to expand variables.

Edit log messages to say e.g "Enabling foo..." instead of
"Enable foo" (the second sounds like we are telling the user
to do something).

Add log message at the end of script, so it shows a message even
if nothing needs to be done on the current run.
The -e makes any command that exits with non-success terminate
the script.

The -u makes any reference to an undefined variable exit with
an error.
@ManfredKarrer ManfredKarrer requested a review from ripcurlx August 25, 2018 21:58
Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK - Thanks for adding the additional checks!

@ripcurlx ripcurlx merged commit cd2a86d into bisq-network:master Aug 28, 2018
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

Successfully merging this pull request may close these issues.

2 participants