Skip to content

Allow SSL configuration from pre-existing keystore to be used#116

Merged
marcuslinke merged 2 commits intodocker-java:masterfrom
magnayn:pr1
Jan 7, 2015
Merged

Allow SSL configuration from pre-existing keystore to be used#116
marcuslinke merged 2 commits intodocker-java:masterfrom
magnayn:pr1

Conversation

@magnayn
Copy link
Copy Markdown
Contributor

@magnayn magnayn commented Dec 19, 2014

Rather than assuming that the relevant keys are present in the local filesystem.

@marcuslinke
Copy link
Copy Markdown
Contributor

@magnayn Currently DockerClientConfigTest fails when docker certs are found in the environment which is needed to run all other integration tests successfully. Maybe you can fix this yourself otherwise it will need some time as i'm busy next days. ;-) Merry XMAS!

accessible on the local disk. This is inconvenient for some clients
who may already have a pre-baked keystore in hand.

Create different options for retrieving the neccessary SSLContext,
so that the two different options can be used.

Signed-off-by: Nigel Magnay <[email protected]>
@magnayn
Copy link
Copy Markdown
Contributor Author

magnayn commented Dec 24, 2014

I've updated 'em - build (on my OS X box) now only fails with the same tests as /master does ;-)

Let me know if it flies for you..

@marcuslinke marcuslinke merged commit 138b11e into docker-java:master Jan 7, 2015
@marcuslinke
Copy link
Copy Markdown
Contributor

@magnayn Thanks!

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.

2 participants