Merge pull request #43 from cyberark/fix-gosec

CNJR-5372: Fix 'nosec' comments
cyberark · Jun 25, 2024 · 1a872cf · 1a872cf
2 parents a71a7cb + 1c32bd8
commit 1a872cf
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/tds.go b/tds.go
@@ -1131,13 +1131,11 @@ initiate_connection:
 			certs.AppendCertsFromPEM([]byte(p.rawCertificate))
 			config.RootCAs = certs
 		}
-		/* #nosec */
 		if p.trustServerCertificate {
-			config.InsecureSkipVerify = true
+			config.InsecureSkipVerify = true // #nosec
 		}
-		/* #nosec */
 		if p.disableVerifyHostname {
-			config.InsecureSkipVerify = true
+			config.InsecureSkipVerify = true // #nosec
 		}
 		config.ServerName = p.hostInCertificate
 		// fix for https://github.com/denisenkom/go-mssqldb/issues/166