Skip to content

Commit f81e6bd

Browse files
committed
fix file closing
1 parent 3fffd50 commit f81e6bd

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

backend/pkg/logger/data/logger.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,16 +164,19 @@ func (sublogger *Logger) Stop() error {
164164
return nil
165165
}
166166

167+
closeErr := error(nil)
167168
for _, file := range sublogger.valueFileSlice {
168169
err := file.Close()
169170
if err != nil {
170-
return loggerHandler.ErrClosingFile{
171+
closeErr = loggerHandler.ErrClosingFile{
171172
Name: Name,
172173
Timestamp: time.Now(),
173174
}
175+
176+
fmt.Println(closeErr.Error())
174177
}
175178
}
176179

177180
fmt.Println("Logger stopped")
178-
return nil
181+
return closeErr
179182
}

backend/pkg/logger/messages/logger.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -143,16 +143,18 @@ func (sublogger *Logger) Stop() error {
143143
return nil
144144
}
145145

146+
closeErr := error(nil)
146147
for _, file := range sublogger.infoIdMap {
147148
err := file.Close()
148149
if err != nil {
149-
fmt.Printf(logger.ErrClosingFile{
150+
closeErr = logger.ErrClosingFile{
150151
Name: Name,
151152
Timestamp: time.Now(),
152-
}.Error())
153+
}
154+
fmt.Println(closeErr.Error())
153155
}
154156
}
155157

156158
fmt.Println("Logger stopped")
157-
return nil
159+
return closeErr
158160
}

0 commit comments

Comments
 (0)