-
Notifications
You must be signed in to change notification settings - Fork 6
tape recorder php-vcr style #4
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
Comments
@jeromegamez: i talked with @adri (one of the maintainers of php-vcr) about this a few weeks ago. he is not sure whether there is much code to share, as php-vcr looks at the raw curl request. @adri: if you have any input, please comment here... psr-7 middleware should allow a much more elegant approach to this than the stream wrappers. in my experience with php-vcr, the challenging part is how to organize data into cassettes (the storage units). organization wise, we maybe should do a separate php-psr7-vcr repository that only cares about psr 7, and a simple plugin in this repository to integrate that system with Httplug. the jobs of recording and restoring are probably too much for a single class - and they could be integrated with psr 7 implementations even when not using Httplug. |
I would like to have a dedicated repository for a php-psr7-vcr – if someone creates one and gives me access, I'll start right away 😇 |
@jeromegamez note your role in the organization 😜 |
given the organisation is called php-http, i think psr7-vcr would be the right name, resulting in |
I think @jeromegamez already set the repo up. |
he did. with the name i suggested. unless you changed the repository name right now :-) |
Nope. Actually we discussed the name in the private chat and we followed the same logic as you did. 😄 |
Can we close this in favor of #3? I think we discussed what needed to be discussed, the rest is up to @jeromegamez 😛 |
Of course :) |
build a plugin to intercept and record / replay responses to requests. inspired by http://php-vcr.github.io/ which does this using a stream wrapper around guzzle calls.
The text was updated successfully, but these errors were encountered: