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

Installer shell script downloads keys over HTTP (not HTTPS) #29

Closed
dturner-tw opened this issue Apr 3, 2014 · 4 comments
Closed

Installer shell script downloads keys over HTTP (not HTTPS) #29

dturner-tw opened this issue Apr 3, 2014 · 4 comments

Comments

@dturner-tw
Copy link

from the installer script:
curl -s http://download.draios.com/DRAIOS-GPG-KEY.public | apt-key add -

This is bad because there's no authentication; anyone could MITM this. HTTPS would solve this problem.

@gianlucaborello
Copy link
Contributor

Good catch!

Problem is we use S3 to host our repository so we can't get HTTPS for download.draios.com, so I'll just move the key to another location or use the full s3.amazonaws.com name.

Thanks for reporting this!

@gianlucaborello
Copy link
Contributor

@gianlucaborello
Copy link
Contributor

So this should be fixed in 98e1970.
I also updated our wiki and website so the entire installer is fetched from HTTPS and a potential MITM is avoided there as well.
Unfortunately the URLs look less pretty but that's S3.

Thanks!

@dturner-tw
Copy link
Author

Awesome! Thanks for the quick response.

On Thu, Apr 3, 2014 at 3:33 PM, Gianluca Borello
notifications@github.comwrote:

So this should be fixed in 98e197098e1970498
.
I also updated our wiki and website so the entire installer is fetched
from HTTPS and a potential MITM is avoided there as well.
Unfortunately the URLs look less pretty but that's S3.

Thanks!

Reply to this email directly or view it on GitHubhttps://github.com//issues/29#issuecomment-39494790
.

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