Apply SslVerifyMode in RedisURI. applySsl(…) #2328

Co-authored-by: Jacob Halsey <[email protected]> Original pull request: #2329
redis · Feb 15, 2023 · 07a0e5b · 07a0e5b
1 parent dcd7085
commit 07a0e5b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/main/java/io/lettuce/core/RedisURI.java b/src/main/java/io/lettuce/core/RedisURI.java
@@ -625,7 +625,7 @@ public void applySsl(RedisURI source) {
         LettuceAssert.notNull(source, "Source RedisURI must not be null");
 
         setSsl(source.isSsl());
-        setVerifyPeer(source.isVerifyPeer());
+        setVerifyPeer(source.getVerifyMode());
         setStartTls(source.isStartTls());
     }
 

diff --git a/src/test/java/io/lettuce/core/RedisURIUnitTests.java b/src/test/java/io/lettuce/core/RedisURIUnitTests.java
@@ -314,6 +314,10 @@ void shouldApplySslSettings() {
         assertThat(target.isSsl()).isTrue();
         assertThat(target.isVerifyPeer()).isFalse();
         assertThat(target.isStartTls()).isTrue();
+
+        source.setVerifyPeer(SslVerifyMode.CA);
+        target.applySsl(source);
+        assertThat(target.getVerifyMode()).isEqualTo(SslVerifyMode.CA);
     }
 
     @Test