Exclude buergertest.ecocare.center from link test

MikeMcC399 commented 1 year ago

This PR addresses the issue reported in https://github.com/corona-warn-app/cwa-website/issues/3415#issuecomment-1455528374. It is an alternative resolution to PR https://github.com/corona-warn-app/cwa-website/pull/3417.
It adds the hostname buergertest.ecocare.center to the list noCheckHosts in cypress/e2e/hybrid/check_links.cy.js, since https://buergertest.ecocare.center/#c734 responds with a 404 (Not Found) error.

$ curl -I https://buergertest.ecocare.center/#c734
HTTP/2 404
x-ua-compatible: IE=edge
content-type: text/html; charset=utf-8
date: Tue, 07 Mar 2023 08:20:57 GMT
server: Apache

MikeMcC399 commented 1 year ago

@brianebeling

This would be a quick, easy and relatively clean fix. I don't think it is worth spending too much time on making a decision on this. There are currently more important issues to be resolved.

brianebeling commented 1 year ago

Brian-J. Ebeling

This would be a quick, easy and relatively clean fix. I don't think it is worth spending too much time on making a decision on this. There are currently more important issues to be resolved.

Hey Mike, its not that I'm spending a lot of time on this, more so that I don't want to be hasty, because there is a lot to do. I already noticed that leads to bad decisions.

I was also waiting for an answer from other participants, if we can make an exception updating the blogs in this case. The answer is yes, we can. However, I'd say we merge this PR and wait for the banner indicating EOL. Then we don't need to change the blogs at all, for now. (unless another link gets hijacked again, in this case I advocate to just remove it)

corona-warn-app / cwa-website

Exclude buergertest.ecocare.center from link test #3418