Thanks for the review @adriancole!

> TLSv1 is suggested here 
> http://www.oracle.com/technetwork/java/javase/documentation/cve-2014-
> 3566-2342133.html and it is not a docker requirement.

W.r.t. this, if this change is not a requirement to support this new API 
version, my vote would be for removing it from this PR and, if desired, 
submitting it separately.

This will allow us to merge this more quickly and get the desired functionality 
it, while keeping the more contentious fix separate and making it much easier 
to revert when/if desired.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/113#issuecomment-64742014

Reply via email to