diff options
author | Francis Lavoie <lavofr@gmail.com> | 2022-06-28 21:23:30 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-06-28 19:23:30 -0600 |
commit | 58e05cab155643c6543892855f373ea8755eb094 (patch) | |
tree | af5fcc1522875bd051ccc95812d15e088479a541 /modules/caddyhttp/reverseproxy/reverseproxy.go | |
parent | 10f85558ead15e119f8e9abd81c8ad55eb865f8b (diff) |
forwardauth: Fix case when `copy_headers` is omitted (#4856)
See https://caddy.community/t/using-forward-auth-and-writing-my-own-authenticator-in-php/16410, apparently it didn't work when `copy_headers` wasn't used. This is because we were skipping adding a handler to the routes in the "good response handler", but this causes the logic in `reverseproxy.go` to ignore the response handler since it's empty. Instead, we can just always put in the `header` handler, even with an empty `Set` operation, it's just a no-op, but it fixes that condition in the proxy code.
Diffstat (limited to 'modules/caddyhttp/reverseproxy/reverseproxy.go')
0 files changed, 0 insertions, 0 deletions