Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feature: allow client redirects via 302 responses (#3130) #3131

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feature: allow client redirects via 302 responses (#3130) #3131

wants to merge 1 commit into from

Commits on Jun 27, 2024

  1. webhook filter: allow client redirects via 302 responses (zalando#3130) 

    This commit changes the behaviour of the webhook filter when a 302 Found
response is recieved from the AuthN/AuthZ endpoint. As a result, it allows
front-end facing (i.e. non-API) traffic to be filtered via the webhook.

Documentation updates and increased test coverage is included.

Incidental: Prevent the webhook client from following redirects from the
AuthN/AuthZ endpoint: during testing I realised that the default `net/http`
behaviour was in use - i.e. redirects were followed.

Signed-off-by: Fergus Morrow <[email protected]>
    @FergusInLondon
    FergusInLondon committed Jun 27, 2024
    Configuration menu
    Copy the full SHA
    fe41415 View commit details
    Browse the repository at this point in the history