DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix deferInLoop error (#10387)

Browse source 
Signed-off-by: z1cheng <imchench@gmail.com>
This commit is contained in:
Chen Chen 2023-09-10 21:48:10 +08:00 committed by GitHub
parent 9666f4ab71
commit 336b1d6fc5
1 changed files with 11 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
internal/nginx/maxmind.go
View file

@ -179,20 +179,19 @@ func downloadDatabase(dbName string) error {
if !strings.HasSuffix(header.Name, mmdbFile) { if !strings.HasSuffix(header.Name, mmdbFile) {
continue continue
} }
return func() error {
outFile, err := os.Create(path.Join(geoIPPath, mmdbFile)) outFile, err := os.Create(path.Join(geoIPPath, mmdbFile))
if err != nil { if err != nil {
return err return err
} }
//nolint:gocritic // TODO: will fix it on a followup PR
defer outFile.Close() defer outFile.Close()
if _, err := io.CopyN(outFile, tarReader, header.Size); err != nil { if _, err := io.CopyN(outFile, tarReader, header.Size); err != nil {
return err return err
} }
return nil return nil
}()
} }
} }