htwkmooc/poseidon
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Change response logging message

Browse Source 
to allow specific error tracking in sentry. See POSEIDON-F
This commit is contained in:
Maximilian Paß
2021-11-26 21:14:27 +01:00
parent a6eaa45097
commit 7c9406cd7a
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/logging/logging.go
View File

@ -66,7 +66,7 @@ func (writer *loggingResponseWriter) Hijack() (net.Conn, *bufio.ReadWriter, erro
return conn, rw, nil return conn, rw, nil
} }
// HTTPLoggingMiddleware returns an http.Handler that logs different information about every request. // HTTPLoggingMiddleware returns a http.Handler that logs different information about every request.
func HTTPLoggingMiddleware(next http.Handler) http.Handler { func HTTPLoggingMiddleware(next http.Handler) http.Handler {
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
start := time.Now().UTC() start := time.Now().UTC()
@ -84,7 +84,7 @@ func HTTPLoggingMiddleware(next http.Handler) http.Handler {
"user_agent": r.UserAgent(), "user_agent": r.UserAgent(),
}) })
if lrw.statusCode >= http.StatusInternalServerError { if lrw.statusCode >= http.StatusInternalServerError {
logEntry.Warn() logEntry.Error("Failing " + path)
} else { } else {
logEntry.Debug() logEntry.Debug()
} }

2
pkg/logging/logging_test.go
View File

@ -28,7 +28,7 @@ func TestHTTPMiddlewareWarnsWhenInternalServerError(t *testing.T) {
HTTPLoggingMiddleware(mockHTTPStatusHandler(500)).ServeHTTP(recorder, request) HTTPLoggingMiddleware(mockHTTPStatusHandler(500)).ServeHTTP(recorder, request)
assert.Equal(t, 1, len(hook.Entries)) assert.Equal(t, 1, len(hook.Entries))
assert.Equal(t, logrus.WarnLevel, hook.LastEntry().Level) assert.Equal(t, logrus.ErrorLevel, hook.LastEntry().Level)
} }
func TestHTTPMiddlewareDebugsWhenStatusOK(t *testing.T) { func TestHTTPMiddlewareDebugsWhenStatusOK(t *testing.T) {