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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR #253 introduces a regression in 1.6 #290

Closed
sagikazarmark opened this issue Jul 24, 2019 · 2 comments
Closed

PR #253 introduces a regression in 1.6 #290

sagikazarmark opened this issue Jul 24, 2019 · 2 comments

Comments

@sagikazarmark
Copy link
Member

Looks like this is another unfortunate regression caused by #253. 馃槥

Details:

https://travis-ci.org/guzzle/guzzle/jobs/558517697#L599

https://github.com/guzzle/guzzle/blob/2297b43/src/RedirectMiddleware.php#L189

@gmponos
Copy link
Member

gmponos commented Jul 29, 2019

I think i have a PR about it...

#284

Isn't this a fix for it?

@gmponos
Copy link
Member

gmponos commented Dec 23, 2019

I guess we can close this...

@gmponos gmponos closed this as completed Dec 23, 2019
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