Skip to content

Commit 1bbdc90

Browse files
authored
fix(logger): call depth for logging events (#97)
1 parent dc17340 commit 1bbdc90

File tree

2 files changed

+14
-11
lines changed

2 files changed

+14
-11
lines changed

.golangci.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,12 @@ linters:
3030

3131
issues:
3232
exclude-rules:
33-
# Exclude some linters from running on tests files.
3433
- path: _test\.go
3534
linters:
3635
- errcheck
3736
- unparam
3837
- prealloc
3938
- funlen
39+
- path: logger/simple_logger.go
40+
linters:
41+
- errcheck

logger/simple_logger.go

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package logger
22

33
import (
4+
"fmt"
45
"log"
56
)
67

@@ -34,7 +35,7 @@ func NewSimpleLogger(logger *log.Logger, level Level) *SimpleLogger {
3435
func (l *SimpleLogger) Trace(msg any) {
3536
if l.Enabled(LevelTrace) {
3637
l.logger.SetPrefix(TracePrefix)
37-
l.logger.Println(msg)
38+
l.logger.Output(3, fmt.Sprint(msg))
3839
}
3940
}
4041

@@ -43,7 +44,7 @@ func (l *SimpleLogger) Trace(msg any) {
4344
func (l *SimpleLogger) Tracef(format string, args ...any) {
4445
if l.Enabled(LevelTrace) {
4546
l.logger.SetPrefix(TracePrefix)
46-
l.logger.Printf(format, args...)
47+
l.logger.Output(3, fmt.Sprintf(format, args...))
4748
}
4849
}
4950

@@ -52,7 +53,7 @@ func (l *SimpleLogger) Tracef(format string, args ...any) {
5253
func (l *SimpleLogger) Debug(msg any) {
5354
if l.Enabled(LevelDebug) {
5455
l.logger.SetPrefix(DebugPrefix)
55-
l.logger.Println(msg)
56+
l.logger.Output(3, fmt.Sprint(msg))
5657
}
5758
}
5859

@@ -61,7 +62,7 @@ func (l *SimpleLogger) Debug(msg any) {
6162
func (l *SimpleLogger) Debugf(format string, args ...any) {
6263
if l.Enabled(LevelDebug) {
6364
l.logger.SetPrefix(DebugPrefix)
64-
l.logger.Printf(format, args...)
65+
l.logger.Output(3, fmt.Sprintf(format, args...))
6566
}
6667
}
6768

@@ -70,7 +71,7 @@ func (l *SimpleLogger) Debugf(format string, args ...any) {
7071
func (l *SimpleLogger) Info(msg any) {
7172
if l.Enabled(LevelInfo) {
7273
l.logger.SetPrefix(InfoPrefix)
73-
l.logger.Println(msg)
74+
l.logger.Output(3, fmt.Sprint(msg))
7475
}
7576
}
7677

@@ -79,7 +80,7 @@ func (l *SimpleLogger) Info(msg any) {
7980
func (l *SimpleLogger) Infof(format string, args ...any) {
8081
if l.Enabled(LevelInfo) {
8182
l.logger.SetPrefix(InfoPrefix)
82-
l.logger.Printf(format, args...)
83+
l.logger.Output(3, fmt.Sprintf(format, args...))
8384
}
8485
}
8586

@@ -88,7 +89,7 @@ func (l *SimpleLogger) Infof(format string, args ...any) {
8889
func (l *SimpleLogger) Warn(msg any) {
8990
if l.Enabled(LevelWarn) {
9091
l.logger.SetPrefix(WarnPrefix)
91-
l.logger.Println(msg)
92+
l.logger.Output(3, fmt.Sprint(msg))
9293
}
9394
}
9495

@@ -97,7 +98,7 @@ func (l *SimpleLogger) Warn(msg any) {
9798
func (l *SimpleLogger) Warnf(format string, args ...any) {
9899
if l.Enabled(LevelWarn) {
99100
l.logger.SetPrefix(WarnPrefix)
100-
l.logger.Printf(format, args...)
101+
l.logger.Output(3, fmt.Sprintf(format, args...))
101102
}
102103
}
103104

@@ -106,7 +107,7 @@ func (l *SimpleLogger) Warnf(format string, args ...any) {
106107
func (l *SimpleLogger) Error(msg any) {
107108
if l.Enabled(LevelError) {
108109
l.logger.SetPrefix(ErrorPrefix)
109-
l.logger.Println(msg)
110+
l.logger.Output(3, fmt.Sprint(msg))
110111
}
111112
}
112113

@@ -115,7 +116,7 @@ func (l *SimpleLogger) Error(msg any) {
115116
func (l *SimpleLogger) Errorf(format string, args ...any) {
116117
if l.Enabled(LevelError) {
117118
l.logger.SetPrefix(ErrorPrefix)
118-
l.logger.Printf(format, args...)
119+
l.logger.Output(3, fmt.Sprintf(format, args...))
119120
}
120121
}
121122

0 commit comments

Comments
 (0)