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

Add first Requests HTTP client implementation #21

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add first Requests HTTP client implementation #21

wants to merge 3 commits into from

Conversation

Afterster
Copy link

My project cannot have strong dependency with cUrl and because of that I created a new transport layer which use Requests project (http://requests.ryanmccue.info/), using cUrl or fsockopen according to what available.

@bshaffer
Copy link
Owner

bshaffer commented Jan 3, 2014

This is interesting... the advantage is Requests uses fsockopen, and this is required for your project?

I think this may be something better suited for your own application. I do not see this being widely useful

@Afterster
Copy link
Author

Yes Requests is required by my project. But there is no much http framework in php (the other popular one I can think of is Guzzle). In the same time it avoids curl strong dependency on your project.
I found that useful for everyone, moreover you already have the http client abstraction layer, it's why I proposed the pull request. Your choice ;)

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.

3 participants