- Issue created by @generalredneck
- πΊπΈUnited States generalredneck
Just noticed that the one I just posted when I was trying to recreate was a curl 6... but I've got a screenshot of a 404 causing it attached.
- @generalredneck opened merge request.
- Status changed to Needs review
almost 2 years ago 3:11am 18 March 2023 - πΊπΈUnited States generalredneck
So I noticed that we weren't really handling the response codes for anything other than 301 and 302... That said, the others probably always just failed like this. I did add some verbiage giving the user what the error message is... Not sure if this is even wanted. Also did a tiny bit of code clean up while I was in there.
- First commit to issue fork.
- Status changed to RTBC
almost 2 years ago 8:11am 19 March 2023 - π©πͺGermany jurgenhaas Gottmadingen
So I noticed that we weren't really handling the response codes for anything other than 301 and 302... That said, the others probably always just failed like this. I did add some verbiage giving the user what the error message is
This is great, thanks a lot.
Not sure if this is even wanted. Also did a tiny bit of code clean up while I was in there.
Absolutely, much appreciated. I only reverted the insertion of unused
$e
variables, since those are not only not required but also create a warning from code style tests since PHP 8.1 -
jurgenhaas β
committed ce19ced8 on 4.1.x authored by
generalredneck β
Issue #3348806 by generalredneck: Core URL Validation Fatals on 404 and...
-
jurgenhaas β
committed ce19ced8 on 4.1.x authored by
generalredneck β
- Status changed to Fixed
almost 2 years ago 8:13am 19 March 2023 Automatically closed - issue fixed for 2 weeks with no activity.