This is an automated email from the ASF dual-hosted git repository.

pivotalsarge pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new df252f1  GEODE-4817: Fix SSLTest. (#1714)
df252f1 is described below

commit df252f13b7f528258536d9830191c8edce6e1e74
Author: Michael "Sarge" Dodge <[email protected]>
AuthorDate: Mon Apr 2 09:07:48 2018 -0700

    GEODE-4817: Fix SSLTest. (#1714)
---
 .../main/java/org/apache/geode/experimental/driver/SocketFactory.java | 2 +-
 .../src/test/java/org/apache/geode/experimental/driver/SSLTest.java   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/geode-experimental-driver/src/main/java/org/apache/geode/experimental/driver/SocketFactory.java
 
b/geode-experimental-driver/src/main/java/org/apache/geode/experimental/driver/SocketFactory.java
index eefc8e1..5afe39f 100644
--- 
a/geode-experimental-driver/src/main/java/org/apache/geode/experimental/driver/SocketFactory.java
+++ 
b/geode-experimental-driver/src/main/java/org/apache/geode/experimental/driver/SocketFactory.java
@@ -132,7 +132,7 @@ public class SocketFactory {
       socket.connect(sockaddr, Math.max(timeout, 0));
       if (socket instanceof SSLSocket) {
         SSLSocket sslSocket = (SSLSocket) socket;
-        sslSocket.setUseClientMode(true); // Should this depend on clientSide?
+        sslSocket.setUseClientMode(true);
         sslSocket.setEnableSessionCreation(true);
         if (timeout > 0) {
           sslSocket.setSoTimeout(timeout);
diff --git 
a/geode-experimental-driver/src/test/java/org/apache/geode/experimental/driver/SSLTest.java
 
b/geode-experimental-driver/src/test/java/org/apache/geode/experimental/driver/SSLTest.java
index 6cd3e7a..a2e1232 100644
--- 
a/geode-experimental-driver/src/test/java/org/apache/geode/experimental/driver/SSLTest.java
+++ 
b/geode-experimental-driver/src/test/java/org/apache/geode/experimental/driver/SSLTest.java
@@ -132,7 +132,7 @@ public class SSLTest {
     driver = new DriverFactory().addLocator("localhost", locatorPort)
         
.setTrustStorePath(CLIENT_TRUST_STORE).setKeyStorePath(CLIENT_KEY_STORE).create();
     Set<String> regionsOnServer = driver.getRegionNames();
-    assertEquals(Collections.singleton("region"), regionsOnServer);
+    assertEquals(Collections.singleton("/region"), regionsOnServer);
     assertTrue(driver.isConnected());
   }
 
@@ -161,7 +161,7 @@ public class SSLTest {
     driver = new DriverFactory().addLocator("localhost", locatorPort)
         .setTrustStorePath(CLIENT_TRUST_STORE).create();
     Set<String> regionsOnServer = driver.getRegionNames();
-    assertEquals(Collections.singleton("region"), regionsOnServer);
+    assertEquals(Collections.singleton("/region"), regionsOnServer);
     assertTrue(driver.isConnected());
   }
 

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to