fix #nosec line placement

backjo · backjo · commit 0fdf538ce864 · 2020-01-26T15:59:32.000-08:00
Signed-off-by: Jonah Back &lt;jonah@jonahback.com&gt;
diff --git a/pkg/config/tlscfg/options.go b/pkg/config/tlscfg/options.go
@@ -43,11 +43,10 @@ func (p Options) Config() (*tls.Config, error) {
 	if err != nil {
 		return nil, errors.Wrap(err, "failed to load CA CertPool")
 	}
-
+	// #nosec G402
 	tlsCfg := &tls.Config{
-		RootCAs:    certPool,
-		ServerName: p.ServerName,
-		// #nosec G402
+		RootCAs:            certPool,
+		ServerName:         p.ServerName,
 		InsecureSkipVerify: p.SkipHostVerify,
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -43,11 +43,10 @@ func (p Options) Config() (*tls.Config, error) {`
`43`	`43`	`if err != nil {`
`44`	`44`	`return nil, errors.Wrap(err, "failed to load CA CertPool")`
`45`	`45`	`}`
`46`		`-`
	`46`	`+ // #nosec G402`
`47`	`47`	`tlsCfg := &tls.Config{`
`48`		`- RootCAs: certPool,`
`49`		`- ServerName: p.ServerName,`
`50`		`- // #nosec G402`
	`48`	`+ RootCAs: certPool,`
	`49`	`+ ServerName: p.ServerName,`
`51`	`50`	`InsecureSkipVerify: p.SkipHostVerify,`
`52`	`51`	`}`
`53`	`52`