Fix G307 warning: Deferring unsafe method "Close" on type "*os.File" (#…

…1154) * fix: check potential error from file.Close * fix(detect): use existing logger
gitleaks · Apr 19, 2023 · 81cf308 · 81cf308
1 parent bd8b145
commit 81cf308
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/detect/detect.go b/detect/detect.go
@@ -154,7 +154,12 @@ func (d *Detector) AddGitleaksIgnore(gitleaksIgnorePath string) error {
 		return err
 	}
 
-	defer file.Close()
+	// https://github.com/securego/gosec/issues/512
+	defer func() {
+		if err := file.Close(); err != nil { 
+			log.Warn().Msgf("Error closing .gitleaksignore file: %s\n", err)
+		}
+    }()
 	scanner := bufio.NewScanner(file)
 
 	for scanner.Scan() {