feat: Add "skip_verify" to Sentinel (#3428)

Same as 3d4310a but for FailoverOptions.

Signed-off-by: Julien Riou <julien@riou.xyz>

Author: jouir

sentinel.go

@@ -272,6 +272,7 @@ func (opt *FailoverOptions) clusterOptions() *ClusterOptions {
 //     URL attributes (scheme, host, userinfo, resp.), query parameters using these
 //     names will be treated as unknown parameters
 //   - unknown parameter names will result in an error
+//   - use "skip_verify=true" to ignore TLS certificate validation
 //
 // Example:
 //
@@ -379,6 +380,10 @@ func setupFailoverConnParams(u *url.URL, o *FailoverOptions) (*FailoverOptions,
 		o.SentinelAddrs = append(o.SentinelAddrs, net.JoinHostPort(h, p))
 	}
 
+	if o.TLSConfig != nil && q.has("skip_verify") {
+		o.TLSConfig.InsecureSkipVerify = q.bool("skip_verify")
+	}
+
 	// any parameters left?
 	if r := q.remaining(); len(r) > 0 {
 		return nil, fmt.Errorf("redis: unexpected option: %s", strings.Join(r, ", "))

sentinel_test.go

@@ -431,6 +431,14 @@ func TestParseFailoverURL(t *testing.T) {
 					ServerName: "localhost",
 				}},
 		},
+		{
+			url: "rediss://localhost:6379/5?master_name=test&skip_verify=true",
+			o: &redis.FailoverOptions{SentinelAddrs: []string{"localhost:6379"}, MasterName: "test", DB: 5,
+				TLSConfig: &tls.Config{
+					ServerName:         "localhost",
+					InsecureSkipVerify: true,
+				}},
+		},
 		{
 			url: "redis://localhost:6379/5?master_name=test&db=2",
 			o:   &redis.FailoverOptions{SentinelAddrs: []string{"localhost:6379"}, MasterName: "test", DB: 2},