This upgrades OkHttp to 2.1.0 and refactors the OkHttp driver to use its native 
methods. This PR could supersede https://github.com/jclouds/jclouds/pull/544 
and fix [JCLOUDS-744](https://issues.apache.org/jira/browse/JCLOUDS-744).

Note that the integration test suite in the OkHttp driver is the same than the 
one for the default HTTP driver, so the coverage is pretty good. I've also 
executed the OpenStack Marconi live tests (Marconi uses the OkHttp driver by 
default) and only one failed but it is unrelated to the change.

This is just an initial implementation of the driver using the 2.1.0 version 
and the OkHttp native API, that should put us in a better place to address the 
commented improvements to the SSL connections. The current implementation is 
based on the default HTTP driver, and there will be room for improvement. In 
any case, it's a nice starting point :)

/cc @adriancole 
You can merge this Pull Request by running:

  git pull https://github.com/nacx/jclouds okhttp-2.1.0

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/617

-- Commit Summary --

  * Upgrade to OkHttp 2.1.0

-- File Changes --

    M apis/ec2/src/test/java/org/jclouds/ec2/internal/BaseEC2ApiMockTest.java 
(6)
    M 
apis/openstack-keystone/src/test/java/org/jclouds/openstack/v2_0/internal/BaseOpenStackMockTest.java
 (2)
    M 
apis/openstack-swift/src/test/java/org/jclouds/openstack/swift/v1/features/ObjectApiMockTest.java
 (9)
    M 
apis/swift/src/test/java/org/jclouds/openstack/swift/blobstore/strategy/internal/SequentialMultipartUploadStrategyMockTest.java
 (2)
    M 
core/src/main/java/org/jclouds/http/internal/BaseHttpCommandExecutorService.java
 (2)
    M 
core/src/test/java/org/jclouds/http/BaseHttpCommandExecutorServiceIntegrationTest.java
 (11)
    M core/src/test/java/org/jclouds/http/BaseMockWebServerTest.java (18)
    M 
drivers/okhttp/src/main/java/org/jclouds/http/okhttp/OkHttpCommandExecutorService.java
 (173)
    M project/pom.xml (2)
    M 
providers/aws-ec2/src/test/java/org/jclouds/aws/ec2/internal/BaseAWSEC2ApiMockTest.java
 (9)
    M 
providers/hpcloud-objectstorage/src/test/java/org/jclouds/hpcloud/objectstorage/internal/BaseHPCloudObjectStorageMockTest.java
 (2)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/617.patch
https://github.com/jclouds/jclouds/pull/617.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/617

Reply via email to