Skip to content

Commit 835d6ef

Browse files
committed
fix: update RedissUsernamePassword test case for TLS 1.2 enforcement
Fix the remaining cluster test failure by updating the RedissUsernamePassword test case to expect MinVersion: tls.VersionTLS12. The test was failing because: - Expected: TLSConfig with MinVersion: 0 (not set) - Actual: TLSConfig with MinVersion: 771 (TLS 1.2) This completes the alignment of all cluster test cases with our security-first approach where all rediss:// URLs automatically enforce TLS 1.2 minimum. All cluster test cases now consistently expect MinVersion: tls.VersionTLS12 for rediss:// URLs, matching the behavior of single client tests.
1 parent 8ff9a76 commit 835d6ef

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

osscluster_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1681,7 +1681,7 @@ EKTcWGekdmdDPsHloRNtsiCa697B2O9IFA==
16811681
}, {
16821682
test: "RedissUsernamePassword",
16831683
url: "rediss://foo:bar@localhost:123?addr=localhost:1234",
1684-
o: &redis.ClusterOptions{Addrs: []string{"localhost:123", "localhost:1234"}, Username: "foo", Password: "bar", TLSConfig: &tls.Config{ServerName: "localhost"}},
1684+
o: &redis.ClusterOptions{Addrs: []string{"localhost:123", "localhost:1234"}, Username: "foo", Password: "bar", TLSConfig: &tls.Config{ServerName: "localhost", MinVersion: tls.VersionTLS12}},
16851685
}, {
16861686
test: "QueryParameters",
16871687
url: "redis://localhost:123?read_timeout=2&pool_fifo=true&addr=localhost:1234",

0 commit comments

Comments
 (0)