Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Update Coop.ch.xml #1188

Merged
merged 6 commits into from
Apr 10, 2015
Merged

Update Coop.ch.xml #1188

merged 6 commits into from
Apr 10, 2015

Conversation

J0WI
Copy link
Contributor

@J0WI J0WI commented Mar 11, 2015

updated domain

updated domain
@@ -14,8 +14,6 @@ Fetch error: http://impo.ch/ => https://www.impo.ch/: Redirect for 'http://www.i
Fetch error: http://www.impo.ch/ => https://www.impo.ch/: Redirect for 'http://www.impo.ch/' missing Location
Fetch error: http://christ-swiss.ch/ => https://www.christ-swiss.ch/: Redirect for 'http://www.christ-swiss.ch/' missing Location
Fetch error: http://www.christ-swiss.ch/ => https://www.christ-swiss.ch/: Redirect for 'http://www.christ-swiss.ch/' missing Location
Fetch error: http://the-body-shop.ch/ => https://www.the-body-shop.ch/: (51, "SSL: no alternative certificate subject name matches target host name 'www.the-body-shop.ch'")
Fetch error: http://www.the-body-shop.ch/ => https://www.the-body-shop.ch/: (51, "SSL: no alternative certificate subject name matches target host name 'www.the-body-shop.ch'")
-->
<ruleset name="Coop" default_off='failed ruleset test'>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change this to 'Needs ruleset tests' or remove it and add the necessary tests.

@J0WI
Copy link
Contributor Author

J0WI commented Mar 23, 2015

Finally! All tests passed.

@jsha
Copy link
Member

jsha commented Mar 24, 2015

There are a test url and a rule for www.coop.ch, but it is not included in the list of target hosts. Please remove them.

This should have been caught by the checker, I filed an issue: https://github.com/hiviah/https-everywhere-checker/issues/

@J0WI
Copy link
Contributor Author

J0WI commented Mar 24, 2015

fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants