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

Use HTTPS everywhere we can. #68

Closed
wants to merge 1 commit into from
Closed

Use HTTPS everywhere we can. #68

wants to merge 1 commit into from

Conversation

technomancy
Copy link
Contributor

Disables two tests which were using an unencrypted HTTP connection to
the Central UK mirrors. I was unable to find any HTTPS mirrors, but if
one exists, it should be used and the tests re-enabled.

@xeqi
Copy link
Contributor

xeqi commented Aug 8, 2014

👎 on removing mirror tests. They get written to a temp cache and never have any code loaded.

@technomancy
Copy link
Contributor Author

The servlet-api artifact is a commonly-used dependency though. Maybe if we tested against an artifact so obscure that it's basically guaranteed never to get consumed from another codebase?

@xeqi
Copy link
Contributor

xeqi commented Aug 8, 2014

The downloaded files are already "basically guaranteed never to get consumed by another codebase". The tests are setup to use a temporary directory as a local cache with :local-repo tmp-local-repo-dir. This also gets deleted each time the tests run through the clear-tmp fixture.

@technomancy
Copy link
Contributor Author

Oh cool; I missed that. Updated the patch to reinstate the test with unencrypted mirrors.

@xeqi
Copy link
Contributor

xeqi commented Aug 8, 2014

I think the mirrors need s/https/http.

@technomancy
Copy link
Contributor Author

Yeah, you caught me in the middle of a few amends/forcepush cycles. =)

@cemerick
Copy link
Collaborator

I'm seeing similar test failures as Travis:

Could not transfer artifact javax.servlet:servlet-api:pom:2.5 from/to central
(https://repo1.maven.org/maven2/): Failed to transfer file:
https://repo1.maven.org/maven2/javax/servlet/servlet-api/2.5/servlet-api-2.5.pom.
Return code is: 503, ReasonPhrase:backend read error.

Browsers and curl can load that URL fine. The other is:

Access denied to: https://uk.maven.org/maven2/javax/servlet/servlet-api/2.5/servlet-api-2.5.pom,
ReasonPhrase:Use https://repo1.maven.org.

As far as Firefox is concerned, the cert for uk.maven.org is invalid (only usable with repo1).

I agree with the sentiment in the subject, but I don't want to lead people into a cert validity / connectivity policy minefield either…

@technomancy
Copy link
Contributor Author

This is covered by #83 now.

@technomancy technomancy deleted the https branch May 18, 2017 22:40
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.

3 participants