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

Google App Engine support #13

Closed
skeemer opened this issue Nov 12, 2014 · 6 comments
Closed

Google App Engine support #13

skeemer opened this issue Nov 12, 2014 · 6 comments

Comments

@skeemer
Copy link
Contributor

skeemer commented Nov 12, 2014

I'm developing an app to run on Google App Engine and ran into a snag. It doesn't have the CURL extension. Guzzle doesn't need it in the newer versions. Can we get it upgraded?

@mfairch
Copy link
Contributor

mfairch commented Nov 12, 2014

The version of Guzzle we use allows us to remain PHP 5.3 compatible. We will discuss a 5.4 branch but for now I'd suggest fork the repo and include that one in your project with the updated dependencies.

@skeemer
Copy link
Contributor Author

skeemer commented Nov 13, 2014

If it's of any interest, here's the fork. I've got it passing the phpunit tests and seems to be fully working.
https://github.com/skeemer/infusionsoft-php/tree/guzzle4

@andrewryno
Copy link
Contributor

The branch looks good to me. It's just up to switching to 5.4 as the minimum requirement. Switching to Guzzle 4 is definitely something I'd like to do since it has a lot of improvements and should help users use third-party adapters (caching, Runscope, logging, etc.).

@andrewryno
Copy link
Contributor

In the meantime you should be able to create a new HttpClient, set that as the default (copy the curl client example), and then use stream_context_create/file_get_contents as Google recommends.

@skeemer
Copy link
Contributor Author

skeemer commented Feb 4, 2015

I haven't tested it yet, but I think we can close this as Google added the CURL extension. https://gae-php-tips.appspot.com/2014/11/15/app-engine-php-1-9-17-pre-release-sdk/

@andrewryno
Copy link
Contributor

It's pre-release, but yeah considering I posted a basic workaround as well for older versions I'm going to go ahead and close it.

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

No branches or pull requests

3 participants