GitHub user spmallette opened a pull request:

    https://github.com/apache/incubator-tinkerpop/pull/293

    TINKERPOP-1269 More Configuration Options for SSL

    Pretty boring pull request. Added more SSL configuration options for 
Gremlin Driver and Server. Existing SSL tests still work and I added a new one 
to validate the overrides worked. Everything still good with:
    
    ```text
    $ mvn clean install -DskipTests && mvn verify -pl gremlin-server 
-DskipIntegrationTests=false 
    ```
    
    Did some manual testing as well with with the `gremlin-server-secure.yaml` 
and the console:
    
    ```text
    gremlin> :remote connect tinkerpop.server conf/remote-secure.yaml
    ==>Connected - localhost/127.0.0.1:8182
    gremlin> :remote console
    ==>All scripts will now be sent to Gremlin Server - 
[localhost/127.0.0.1:8182] - type ':remote console' to return to local mode
    gremlin> 1+1
    ==>2
    ```
    
    VOTE +1

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/incubator-tinkerpop TINKERPOP-1269

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-tinkerpop/pull/293.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #293
    
----
commit 57416163be0470d0dcd124bebc12efe325578805
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2016-04-19T17:53:35Z

    Added additional SSL configuration options for Gremlin Driver.
    
    Specifically added: keyPassword, keyFile and keyCertChainFile.

commit cfa95c8a29b4e27c9fd8e047a5e3631f915de005
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2016-04-19T18:02:12Z

    Added docs for new SSL settings on the driver side.

commit 9996066189686812befdeb69e33ef2e9a8a6b9b7
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2016-04-20T17:50:01Z

    Added ability to pass an SslContext to the driver.
    
    This will be useful if the user needs more complex ssl configuration than 
the Cluster builder allows.

commit 75e875700201013df7b2323a7a11907c0b9ce500
Author: Stephen Mallette <sp...@genoprime.com>
Date:   2016-04-20T20:13:24Z

    Added configuration for overriding the Gremlin Server SSL config with 
SslContext.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to