diff options
author | Robin Lambertz <github@roblab.la> | 2020-02-20 23:00:30 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-02-20 15:00:30 -0700 |
commit | e3591009dc8cf386704f8671c54b0f97c3f75b8c (patch) | |
tree | a85939db46988e8bbeb2ff3d82a7e932a6ea65ef /modules/caddyhttp/encode/encode_test.go | |
parent | 30c14084abda8565215972fdffc5bbc41708b32b (diff) |
caddyhttp: Add handler for unhandled errors in errorChain (#3063)
* Add handler for unhandled errors in errorChain
Currently, when an error chain is defined, the default error handler is
bypassed entirely - even if the error chain doesn't handle every error.
This results in pages returning a blank 200 OK page.
For instance, it's possible for an error chain to match on the error
status code and only handle a certain subtype of errors (like 403s). In
this case, we'd want any other errors to still go through the default
handler and return an empty page with the status code.
This PR changes the "suffix handler" passed to errorChain.Compile to
set the status code of the response to the error status code.
Fixes #3053
* Move the errorHandlerChain middleware to variable
* Style fix
Diffstat (limited to 'modules/caddyhttp/encode/encode_test.go')
0 files changed, 0 insertions, 0 deletions