Merge pull request #54 from dell/tls-update

Fixing tls issues
dell · Aug 28, 2024 · 348bace · 348bace
2 parents 10cd868 + b5c3092
commit 348bace
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/api/restclient.go b/api/restclient.go
@@ -143,7 +143,7 @@ func New(_ context.Context, host string, opts ClientOptions, debug bool) (Client
 		c.http.Timeout = opts.Timeout
 	}
 
-	if opts.Insecure {
+	if opts.Insecure { // #nosec G402
 		c.http.Transport = &http.Transport{
 			TLSClientConfig: &tls.Config{
 				InsecureSkipVerify: true,
@@ -160,6 +160,7 @@ func New(_ context.Context, host string, opts ClientOptions, debug bool) (Client
 				RootCAs:            pool,
 				InsecureSkipVerify: false,
 				CipherSuites:       util.GetSecuredCipherSuites(),
+				MinVersion:         tls.VersionTLS12,
 			},
 		}
 	}