internetstandards / Internet.nl

Internet standards compliance test suite
https://internet.nl
176 stars 37 forks source link

Add remark to test explanation "Same website on IPv6 and IPv4" subtest on expecting 20x and 30x HTTP status code #1226

Open baknu opened 9 months ago

baknu commented 9 months ago

https://internet.nl/site/403.returnco.de/2571124/

https://internet.nl/site/200.returnco.de/2571129/

https://internet.nl/site/201.returnco.de/2571128/

https://internet.nl/site/301.returnco.de/2571140/

https://internet.nl/site/302.returnco.de/2571142/

bwbroersma commented 9 months ago

Note: 3xx to a 5xx is also a failure (just had such a case). Here is the relevant code from the simhash: https://github.com/internetstandards/Internet.nl/blob/7bb8f53b97ffe467080884250bc0a81f029a978c/checks/tasks/ipv6.py#L580-L581 which uses the http client wrapper around Python requests which follows redirects.

bwbroersma commented 9 months ago

Behavior changed, now ⚠️ on 4xx and 5xx, see issue #1267 (fix/PR #1271), that should still be explained.