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/logging | |
| 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/logging')
0 files changed, 0 insertions, 0 deletions