You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Ignasi Barrera <no...@github.com> on 2014/01/14 13:44:40 UTC
[jclouds] BACKPORT: Added OkHttp driver to support modern HTTP verbs
(#251)
Backport https://github.com/jclouds/jclouds/pull/232 to 1.7.x
/cc @everett-toews
You can merge this Pull Request by running:
git pull https://github.com/nacx/jclouds okhttp-17
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/251
-- Commit Summary --
* Added OkHttp driver to support modern HTTP verbs
-- File Changes --
M apis/openstack-keystone/pom.xml (1)
M core/pom.xml (5)
M core/src/main/java/org/jclouds/http/internal/JavaUrlHttpCommandExecutorService.java (67)
M core/src/test/java/org/jclouds/http/BackoffLimitedRetryJavaTest.java (155)
M core/src/test/java/org/jclouds/http/BaseHttpCommandExecutorServiceIntegrationTest.java (648)
A core/src/test/java/org/jclouds/http/BaseMockWebServerTest.java (130)
M core/src/test/java/org/jclouds/http/IntegrationTestAsyncClient.java (4)
M core/src/test/java/org/jclouds/http/IntegrationTestClient.java (4)
M core/src/test/java/org/jclouds/http/JavaUrlHttpCommandExecutorServiceIntegrationTest.java (4)
M core/src/test/java/org/jclouds/http/handlers/BackoffLimitedRetryHandlerTest.java (11)
M drivers/apachehc/src/test/java/org/jclouds/http/apachehc/ApacheHCHttpCommandExecutorServiceTestDisabled.java (2)
M drivers/gae/pom.xml (5)
M drivers/gae/src/test/java/org/jclouds/gae/AsyncGaeHttpCommandExecutorServiceIntegrationTest.java (157)
A drivers/okhttp/README.md (15)
A drivers/okhttp/pom.xml (64)
A drivers/okhttp/src/main/java/org/jclouds/http/okhttp/OkHttpCommandExecutorService.java (98)
A drivers/okhttp/src/main/java/org/jclouds/http/okhttp/config/OkHttpCommandExecutorServiceModule.java (43)
A drivers/okhttp/src/test/java/org/jclouds/http/okhttp/OkHttpCommandExecutorServiceTest.java (156)
A drivers/okhttp/src/test/resources/test.jks (0)
M drivers/pom.xml (1)
M project/pom.xml (6)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/251.patch
https://github.com/jclouds/jclouds/pull/251.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/251
Re: [jclouds] BACKPORT: Added OkHttp driver to support modern HTTP
verbs (#251)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #506](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/506/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/251#issuecomment-32264843
Re: [jclouds] BACKPORT: Added OkHttp driver to support modern HTTP
verbs (#251)
Posted by Andrew Phillips <no...@github.com>.
Skimmed over this as I assume it's mainly a backport of already reviewed code. Bearing that in mind, +1 - looks good to me. Thanks, @nacx!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/251#issuecomment-32270637
Re: [jclouds] BACKPORT: Added OkHttp driver to support modern HTTP
verbs (#251)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #970](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/970/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/251#issuecomment-32264794
Re: [jclouds] BACKPORT: Added OkHttp driver to support modern HTTP
verbs (#251)
Posted by Ignasi Barrera <no...@github.com>.
Yes, it is exactly the code in master, only changing the version in the jclouds-okhttp driver pom to match the branch version. No other changes were required to make it compile, so... good! :)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/251#issuecomment-32271594