-
Notifications
You must be signed in to change notification settings - Fork 308
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
Replace our HTTP client abstraction by Guzzle #95
Comments
👍 |
Note: the Guzzle client to be used should be passable via the constructor as optional argument, so everyone can keep using their Guzzle middleware. |
Could someone provide me with a list of benefits to end users of the library that using guzzle will bring? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you'd like this issue to stay open please leave a comment indicating how this issue is affecting you. Thank you. |
This was released in version 6. |
It's not worth hitting the same issues when Guzzle already did all the work.
Bonus point for creating a Pusher mock around Guzzle's
Fixes #87
The text was updated successfully, but these errors were encountered: