-
Notifications
You must be signed in to change notification settings - Fork 685
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.
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
Update HTTPRequestRedirectPolicy StatusCode to use RedirectResponseCode #6789
base: main
Are you sure you want to change the base?
Update HTTPRequestRedirectPolicy StatusCode to use RedirectResponseCode #6789
Conversation
Hi @billyjs! Welcome to our community and thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Contour better. You can also join us on our mailing list and in our channel in the Kubernetes Slack Workspace |
Hi @billyjs! You can run |
Use RedirectResponseCode instead of just 301/302 for HTTPRequestRedirectPolicy StatusCode Fixes projectcontour#6788 Signed-off-by: Billy Schulze <[email protected]>
8f39293
to
4af6a40
Compare
@tsaarni what is the release cadence for this project? |
Generally, we have aimed for quarterly releases but that schedule hasn't always been strictly followed. By the way, I noticed there's a minor issue (#6791) that needs to be resolved before moving forward with this PR |
Thanks, i was trying to figure out why it was not updating that file. |
Signed-off-by: Billy Schulze <[email protected]>
65b4c13
to
f4e4e33
Compare
Signed-off-by: Billy Schulze <[email protected]>
To fix #6788
Allow 303, 307, 308 response status codes when using the request redirect policy.