WIP more WS_Overflowed() checks
Re #4232
Adding earlier overflow checks is pretty easy if we use VCL failure, but so far we had centralized the overflow handling on the client side in the vtr_deliver_f, sending downstream a 500 error and not a 503.
So my main question is: If we want consistency here, can we settle on simply handling workspace overflows as VCL failures? Or do we really want the separate error code still? If we do, improving the overflow detection on the backend side might become a bit more involved, because right now the backend side error just turns into a 503 on the client side.
Note that this PR is really for discussing the design questions, it does not pass the test suite because of the changed error code.
To make progress here, I'd suggest the following:
- turn all VCL and out-of-workspace related errors into
500because this is really what they are. 503 makes specific reference to a temporary overload or scheduled maintenance, but if we run into a VCL failure or out-of-workspace, that is likely to persist for the next request. - differentiate other error conditions. I think we have also more tickets in this area...
bugwash: turn it all into 500, update PR, write doc