Merge pull request #262 from atc0005/i261-ignore-false-positive-gosec…

…-linting-error Ignore false-positive gosec G307 linting errors
atc0005 · Nov 10, 2021 · 5feb4d0 · 5feb4d0
2 parents 58f917e + a1f1a1c
commit 5feb4d0
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/internal/config/config.go b/internal/config/config.go
@@ -246,6 +246,10 @@ func NewConfig() (*Config, error) {
 				err,
 			)
 		}
+
+		// #nosec G307
+		// Believed to be a false-positive from recent gosec release
+		// https://github.com/securego/gosec/issues/714
 		defer func() {
 			if err := fh.Close(); err != nil {
 				// Ignore "file already closed" errors

diff --git a/internal/files/process.go b/internal/files/process.go
@@ -593,6 +593,10 @@ func appendToFile(entry fileEntry, tmpl *template.Template, filename string, per
 			opErr,
 		)
 	}
+
+	// #nosec G307
+	// Believed to be a false-positive from recent gosec release
+	// https://github.com/securego/gosec/issues/714
 	defer func(filename string) {
 		if err := f.Close(); err != nil {
 			// Ignore "file already closed" errors

diff --git a/internal/fileutils/contains.go b/internal/fileutils/contains.go
@@ -53,6 +53,10 @@ func HasLine(searchTerm string, ignorePrefix string, filename string) (bool, err
 			err,
 		)
 	}
+
+	// #nosec G307
+	// Believed to be a false-positive from recent gosec release
+	// https://github.com/securego/gosec/issues/714
 	defer func() {
 		if err := f.Close(); err != nil {
 			// Ignore "file already closed" errors