[dunglas]

go_read_post discarded read errors, so PHP saw a truncated php://input when Caddy's request_body max_size (or any http.MaxBytesReader) was exceeded. The request is now rejected with 413 if the error is a *http.MaxBytesError and response headers have not been sent yet. This works in classic and worker mode.

Fixes #2781

[AlliBalliBaba]

I kind of dislike the silent exit TBH. I could imagine it leading to issues that are hard to debug.

Throwing an error instead feels a bit better, but it would force every application to handle that error.

What I'd actually prefer instead is maybe just buffering all chunked body requests per default so they never reach PHP before fully buffered. Makes it more consistent with what the CGI protocol expects. Similar idea to #2541.