Ignore false-positive gosec G307 linting errors

Issues reported after upgrading golangci-lint to v1.43.0. gosec was updated in that version from v2.8.1 to v2.9.1. refs GH-70 refs golangci/golangci-lint#2299
atc0005 · Nov 30, 2021 · 7cfcb25 · 7cfcb25
1 parent a9e9705
commit 7cfcb25
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/activefile/activefile.go b/activefile/activefile.go
@@ -138,6 +138,10 @@ func (afr activeFileReader) filterEntries(validPrefixes []string) ([]ezproxy.Fil
 	if err != nil {
 		return nil, fmt.Errorf("func filterEntries: error encountered opening file %q: %w", afr.Filename, 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

diff --git a/auditlog/auditlog.go b/auditlog/auditlog.go
@@ -161,6 +161,10 @@ func (alr auditLogReader) AllSessionEntries() (SessionEntries, error) {
 	if err != nil {
 		return nil, fmt.Errorf("func AllSessionEntries: error encountered opening file %q: %w", alr.Filename, 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