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

[Archive.is] Update ruleset #5886

Merged
merged 2 commits into from Aug 2, 2016

Conversation

Projects
None yet
2 participants
@fuglede
Contributor

fuglede commented Jul 31, 2016

After having had a couple of different issues with this ruleset, cf. in particular #5805 and #4075.

I prompted the webmaster for what would be a proper solution moving forwards. On their request, this avoids redirecting anything to https://archive.is/ but ensures that the relevant connections are still upgrading by using archive.fo instead. Indeed, they considered disabling the option to use secure connections to archive.is but were happy with still providing the option in this slightly round-about fashion.

This closes #5805.

[Archive.is] Update ruleset
After having had a couple of different issues with this ruleset, cf. in particular

    #5805 and
    #4075

I prompted the webmaster for what would be a proper solution moving forwards. On their request, this avoids redirecting anything to https://archive.is/ but ensures that the relevant connections are still upgrading by using archive.fo instead.
@J0WI

This comment has been minimized.

Show comment
Hide comment
@J0WI

J0WI Aug 1, 2016

Collaborator

Can you comment that in the rule that we don't forget this in future?

Collaborator

J0WI commented Aug 1, 2016

Can you comment that in the rule that we don't forget this in future?

@J0WI J0WI added the ruleset-bug label Aug 1, 2016

@fuglede

This comment has been minimized.

Show comment
Hide comment
@fuglede

fuglede Aug 2, 2016

Contributor

Sure.

Contributor

fuglede commented Aug 2, 2016

Sure.

@J0WI J0WI merged commit 2bcd01a into EFForg:master Aug 2, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@fuglede fuglede deleted the fuglede:archive-patch branch Aug 3, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment