Merge pull request #2 from hirose31/gosec

Ignore gosec G307
hirose31 · Oct 26, 2021 · da37002 · da37002
2 parents e7033aa + 9a228e8
commit da37002
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -37,7 +37,7 @@ $(GOBIN)/golint:
 	@cd && go get golang.org/x/lint/golint
 
 $(GOBIN)/gosec:
-	@cd && go get github.com/securego/gosec/v2/cmd/gosec@v2.8.1
+	@cd && go get github.com/securego/gosec/v2/cmd/gosec
 
 .PHONY: cross
 cross: $(GOBIN)/goxz ## build for cross platforms

diff --git a/pkg/m/s3.go b/pkg/m/s3.go
@@ -247,6 +247,7 @@ func (s3m S3Model) Download(object s3types.Object) (n int64, err error) {
 	if err != nil {
 		return 0, err
 	}
+	/* #nosec G307 */
 	defer func() {
 		if err := fp.Close(); err != nil {
 			panic(err)