Log request URI rather than path.
This commit is contained in:
parent
ca22f54c89
commit
11def45666
@ -18,5 +18,10 @@ func Ctx(ctx echo.Context) *slog.Logger {
|
||||
return l
|
||||
}
|
||||
|
||||
return Default()
|
||||
}
|
||||
|
||||
// Default returns the default logger
|
||||
func Default() *slog.Logger {
|
||||
return slog.Default()
|
||||
}
|
||||
|
@ -59,13 +59,7 @@ func LogRequest() echo.MiddlewareFunc {
|
||||
"latency", stop.Sub(start).String(),
|
||||
)
|
||||
|
||||
msg := fmt.Sprintf("%s %s", req.Method, func() string {
|
||||
p := req.URL.Path
|
||||
if p == "" {
|
||||
p = "/"
|
||||
}
|
||||
return p
|
||||
}())
|
||||
msg := fmt.Sprintf("%s %s", req.Method, req.URL.RequestURI())
|
||||
|
||||
if res.Status >= 500 {
|
||||
sub.Error(msg)
|
||||
|
@ -77,7 +77,7 @@ func TestLogRequest(t *testing.T) {
|
||||
h := new(mockLogHandler)
|
||||
|
||||
exec := func() {
|
||||
ctx, _ := tests.NewContext(c.Web, "http://test.localhost/abc")
|
||||
ctx, _ := tests.NewContext(c.Web, "http://test.localhost/abc?d=1&e=2")
|
||||
logger := slog.New(h).With("previous", "param")
|
||||
log.Set(ctx, logger)
|
||||
ctx.Request().Header.Set("Referer", "ref.com")
|
||||
@ -101,7 +101,7 @@ func TestLogRequest(t *testing.T) {
|
||||
assert.Equal(t, "5", h.GetAttr("bytes_out"))
|
||||
assert.NotEmpty(t, h.GetAttr("latency"))
|
||||
assert.Equal(t, "INFO", h.level)
|
||||
assert.Equal(t, "GET /abc", h.msg)
|
||||
assert.Equal(t, "GET /abc?d=1&e=2", h.msg)
|
||||
|
||||
statusCode = 500
|
||||
exec()
|
||||
|
Loading…
Reference in New Issue
Block a user