The Wayback Machine - https://web.archive.org/web/20210603131641/https://github.com/dotnet/aspnetcore/pull/33246
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.

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

RedirectHandler follows HTTP status code 303 See other #33246

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

Conversation

@ambrosmiroslav
Copy link

@ambrosmiroslav ambrosmiroslav commented Jun 3, 2021

Addresses #33218.

The HttpClient created by a WebApplicationFactory uses RedirectHandler object to handle the HTTP redirections.
But RedirectHandler didn't recognize status code 303 as the redirection status so the result of the HttpClient request was the redirection itself instead of the redirected content.

Missing HTTP status code for 303 See other was added.

Missing HTTP status code from the 3xx class was added.
@dnfadmin
Copy link

@dnfadmin dnfadmin commented Jun 3, 2021

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

ambrosmiroslav sign now
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants