test: update test now that behavior of erp5.com changed
apparently with caddy we redirect http->https with a 301
It seems that with our caddy frontend behaviour of
https-onlychanged, we now redirect with 301 and no longer 302 ( here ). According to https://caddyserver.com/docs/redir
redirby default makes permanent redirections.
If you ask me, I feel it's fine to use 301 here. Also, I believe this
https-onlyshould default to true. These days using http is really an exception.
I prefer to do so, I will check if I missed assertion and add one.
So as I see in the test, the assertion for http code was missing in test_https_only, that's why the test passed on
So I am going to make another MR adding assertion for 302 and fixing it.
Edit: I noted that
test_type_redirectwas also missing HTTP code assertion.
closedToggle commit list