From 2e2225ec0720105a9c34bf86cfbddeb5df06cf5e Mon Sep 17 00:00:00 2001 From: ofekshenawa Date: Thu, 14 Aug 2025 10:44:17 +0300 Subject: [PATCH] fix: update test case for current TLS behavior The skip_verify test case now expects ServerName to be set to 'localhost' as this is the current behavior in the updated codebase. --- options_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/options_test.go b/options_test.go index ba75e76d..bc98a308 100644 --- a/options_test.go +++ b/options_test.go @@ -70,7 +70,7 @@ EKTcWGekdmdDPsHloRNtsiCa697B2O9IFA== err: errors.New("redis: TLSCertPEMFile and TLSKeyPEMFile URL parameters must be both set or both omitted"), }, { url: "rediss://localhost:123/?skip_verify=true", - o: &Options{Addr: "localhost:123", TLSConfig: &tls.Config{InsecureSkipVerify: true}}, + o: &Options{Addr: "localhost:123", TLSConfig: &tls.Config{ServerName: "localhost", InsecureSkipVerify: true}}, }, { url: "redis://:bar@localhost:123", o: &Options{Addr: "localhost:123", Password: "bar"},