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

HTTP 307 (Temporary Redirect) and HTTP 308 (Permanent Redirect) redirects are not followed (in 4.x) #821

Closed
matthewhorridge opened this issue Feb 7, 2019 · 3 comments

Comments

@matthewhorridge
Copy link
Contributor

HTTP 307 (Temporary Redirect) and HTTP 308 (Permanent Redirect) redirects are not followed (in 4.x)

HTTP 301, 302 and 303 are followed. Please can the other codes be handled? Relevant place in the code is here.

@ignazio1977
Copy link
Contributor

version 6 uses okhttp which handles this by default

@matthewhorridge
Copy link
Contributor Author

Thanks for fixing!

@matthewhorridge
Copy link
Contributor Author

Just been looking at this.... it's revealed another problem (at least in v4). If there's a redirect and the protocol doesn't change then the redirect doesn't happen. See

if (!originalProtocol.equals(newProtocol)) {

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

No branches or pull requests

2 participants