Skip to content

Commit bcd058a

Browse files
committed
update slog adapter to be more robust
1 parent 26fa59d commit bcd058a

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

pkg/observability/log/slog_adapter.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,9 @@ func (l *SlogLogger) Error(msg string, fields ...slog.Attr) {
4444

4545
func (l *SlogLogger) Fatal(msg string, fields ...slog.Attr) {
4646
l.Log(context.Background(), FatalLevel, msg, fields...)
47-
panic("fatal log called")
47+
// Sync to ensure the log message is flushed before panic
48+
_ = l.Sync()
49+
panic("fatal: " + msg)
4850
}
4951

5052
func (l *SlogLogger) WithFields(fields ...slog.Attr) Logger {
@@ -60,6 +62,9 @@ func (l *SlogLogger) WithFields(fields ...slog.Attr) Logger {
6062
}
6163

6264
func (l *SlogLogger) WithError(err error) Logger {
65+
if err == nil {
66+
return l
67+
}
6368
return &SlogLogger{
6469
logger: l.logger.With("error", err.Error()),
6570
level: l.level,

0 commit comments

Comments
 (0)