1berry/docker-cli
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6113 from thaJeztah/config_suppress_err

Browse Source 
cli/config/configfile: explicitly ignore error
This commit is contained in:
Paweł Gronowski 2025-05-28 12:08:37 +00:00 committed by GitHub
commit a007d1ae24
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cli/config/configfile/file.go
View File

@ -152,7 +152,8 @@ func (configFile *ConfigFile) Save() (retErr error) {
return err return err
} }
defer func() { defer func() {
temp.Close() // ignore error as the file may already be closed when we reach this.
_ = temp.Close()
if retErr != nil { if retErr != nil {
if err := os.Remove(temp.Name()); err != nil { if err := os.Remove(temp.Name()); err != nil {
logrus.WithError(err).WithField("file", temp.Name()).Debug("Error cleaning up temp file") logrus.WithError(err).WithField("file", temp.Name()).Debug("Error cleaning up temp file")