Skip to content

[HttpClient] rewind stream when using Psr18Client #32689

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

Merged
merged 1 commit into from
Jul 24, 2019

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Jul 23, 2019

Q A
Branch? 4.3
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets symfony/symfony-docs#11996
License MIT
Doc PR -

This is not a bug fix technically but just how PSR-7 works.
I'm glad we did not make it a first-class thing in Symfony.
This makes it a bit more practicable if it can be...

@fabpot
Copy link
Member

fabpot commented Jul 24, 2019

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 7f4362b into symfony:4.3 Jul 24, 2019
fabpot added a commit that referenced this pull request Jul 24, 2019
…-grekas)

This PR was merged into the 4.3 branch.

Discussion
----------

[HttpClient] rewind stream when using Psr18Client

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | symfony/symfony-docs#11996
| License       | MIT
| Doc PR        | -

This is not a bug fix technically but just how PSR-7 works.
I'm glad we did not make it a first-class thing in Symfony.
This makes it a bit more practicable if it can be...

Commits
-------

7f4362b [HttpClient] rewind stream when using Psr18Client
fabpot added a commit that referenced this pull request Jul 24, 2019
…grekas)

This PR was merged into the 4.4 branch.

Discussion
----------

[HttpClient] rewind streams created from strings

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Follow up of #32689 for 4.4

Commits
-------

33ed4e4 [HttpClient] rewind streams created from strings
@nicolas-grekas nicolas-grekas deleted the hc-psr18-fix branch July 24, 2019 10:47
@fabpot fabpot mentioned this pull request Jul 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants