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

[HttpClient] fix race condition when reading response with informational status #33691

Merged
merged 1 commit into from Sep 26, 2019

Conversation

@nicolas-grekas
Copy link
Member

nicolas-grekas commented Sep 24, 2019

Q A
Branch? 4.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Spotted by some transiently failing jobs on Travis.

@nicolas-grekas nicolas-grekas force-pushed the nicolas-grekas:hc-info-race branch from 06d702d to 450c3c4 Sep 24, 2019
nicolas-grekas added a commit that referenced this pull request Sep 26, 2019
… informational status (nicolas-grekas)

This PR was merged into the 4.3 branch.

Discussion
----------

[HttpClient] fix race condition when reading response with informational status

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Spotted by some transiently failing jobs on Travis.

Commits
-------

450c3c4 [HttpClient] fix race condition when reading response with informational status
@nicolas-grekas nicolas-grekas merged commit 450c3c4 into symfony:4.3 Sep 26, 2019
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
fabbot.io Your code looks good.
Details
@nicolas-grekas nicolas-grekas deleted the nicolas-grekas:hc-info-race branch Sep 30, 2019
@fabpot fabpot mentioned this pull request Oct 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.