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

build: specify a dist.ipfs.io hash #2711

Merged
merged 1 commit into from
May 17, 2016
Merged

build: specify a dist.ipfs.io hash #2711

merged 1 commit into from
May 17, 2016

Conversation

ghost
Copy link

@ghost ghost commented May 17, 2016

build: specify a dist.ipfs.io hash

Right now bin/dist_get trusts whatever DNS resolver is active.
This change enables it to verify what it downloads.
Verification itself remains TODO.

This will work best with a unixfs-hash tool which is TODO too.
It'd just do the equivalent of ipfs add -n -r <dirOrFile>.

License: MIT
Signed-off-by: Lars Gierth [email protected]

Right now bin/dist_get trusts whatever DNS resolver is active.
This change enables it to verify what it downloads.
Verification itself remains TODO.

This will work best with a unixfs-hash tool which is TODO too.
It'd just do the equivalent of `ipfs add -n -r <dirOrFile>`.

License: MIT
Signed-off-by: Lars Gierth <[email protected]>
@ghost ghost added topic/tools Topic tools topic/security Topic security labels May 17, 2016
@ghost
Copy link
Author

ghost commented May 17, 2016

Also enables SSL there

@jbenet
Copy link
Member

jbenet commented May 17, 2016

Very nice! LGTM

@jbenet
Copy link
Member

jbenet commented May 17, 2016

@whyrusleeping thoughts?

@whyrusleeping
Copy link
Member

@lgierth this LGTM, mind changing this at https://github.com/whyrusleeping/dist_get too?

@ghost
Copy link
Author

ghost commented May 17, 2016

@ghost ghost added the RFM label May 17, 2016
@whyrusleeping whyrusleeping merged commit 1e717b8 into master May 17, 2016
@whyrusleeping whyrusleeping deleted the dist-root branch May 17, 2016 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFM topic/security Topic security topic/tools Topic tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants