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

some packages are downloaded via HTTP ? #73

Open
amkhlv opened this issue Oct 15, 2020 · 7 comments
Open

some packages are downloaded via HTTP ? #73

amkhlv opened this issue Oct 15, 2020 · 7 comments

Comments

@amkhlv
Copy link

amkhlv commented Oct 15, 2020

When calling raco pkg install rash, it seems that some packages, such as overeasy and mcfly are downloaded via plain HTTP, no SSL... Could this open a possibility of MITM attack?

@willghatch
Copy link
Owner

willghatch commented Oct 15, 2020 via email

@willghatch
Copy link
Owner

I've gone ahead and split out the demo code into a separate package. This fixes the issue unless you also install that new package. Since the demos should mostly be used for understanding and writing your own code (they explicitly promise no stability), I don't mind as much that the problem lives on there. However, perhaps if you contact the author of those packages he will consider at least adding https to his server or to the package URLs.

@amkhlv
Copy link
Author

amkhlv commented Oct 15, 2020

Thank you ! But, I am surprised that raco pkg allows this to happen... Should I file a bug against Raco?
Also, notice that Neil Van Dyke does have HTTPS on his website. Is it that only packages are over HTTP? Could it be an issue with Raco ?

@amkhlv
Copy link
Author

amkhlv commented Oct 15, 2020

And also, csv-reading ...

@willghatch
Copy link
Owner

willghatch commented Oct 15, 2020 via email

@mflatt
Copy link

mflatt commented Oct 16, 2020

It's rare for packages to be accessed via HTTP, because most are on GitHub, and the package manager rewrites http:// GitHub paths to use the Git protocol over SSL. Aside from Neil's many packages, I see only five other packages that use HTTP.

Making raco pkg rewrite all HTTP to HTTPS by default (unless the behavior is overridden with, say, an environment variable) is probably a good idea, and I think that would make a good issue at https://github.com/racket/racket . With that change, at least three packages registered as HTTP won't install; requring a user to specifically opt in to HTTP seems appropriate, but I'm not sure how easy it will be to detect a certificate or connection failure and recommend a way to opt in.

@amkhlv
Copy link
Author

amkhlv commented Oct 16, 2020

OK, I submitted racket/racket#3443

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

3 participants