Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix stream wrapper (tweaked #193)#230

Open
snelg wants to merge 4 commits intophp-vcr:masterfrom
snelg:fix-stream-wrapper
Open

Fix stream wrapper (tweaked #193)#230
snelg wants to merge 4 commits intophp-vcr:masterfrom
snelg:fix-stream-wrapper

Conversation

@snelg
Copy link

@snelg snelg commented Nov 7, 2017

Context

Tweaked version of #193 to pass all tests and Travis builds

What has been done

Notes

See #193 for other details

markstory and others added 4 commits November 7, 2017 10:02
When using the stream_* methods some clients expect that there is an
internal key of `headers`. By emulating this key VCR can be compatible
with more clients. I encountered this issue when trying to use VCR with
the CakePHP HTTP Client which uses the stream API.
@snelg snelg changed the title Fix stream wrapper Fix stream wrapper (tweaked #193) Nov 7, 2017
@markstory
Copy link

Thanks @snelg 👏

@markstory markstory mentioned this pull request Jan 4, 2018
@cierzniak cierzniak mentioned this pull request Feb 20, 2018
@killua99
Copy link

What should we do to get this code in master and a new tag ?

@Spriz
Copy link

Spriz commented Apr 4, 2022

Hi there - is there any plans if this fix would go into master?

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

Successfully merging this pull request may close these issues.

4 participants