From 5e7d1c6f9e6cdeea11848dabc285181f604f179b Mon Sep 17 00:00:00 2001 From: Doug Lauder Date: Fri, 8 Sep 2023 18:24:34 -0400 Subject: [PATCH] MM-37511 Add userid to http logs (#24516) * include user_id in http logs --- server/channels/web/handlers.go | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/server/channels/web/handlers.go b/server/channels/web/handlers.go index 9a8902e649..05b0e87982 100644 --- a/server/channels/web/handlers.go +++ b/server/channels/web/handlers.go @@ -149,6 +149,11 @@ func (h Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) { appInstance := app.New(app.ServerConnector(h.Srv.Channels())) + c := &Context{ + AppContext: &request.Context{}, + App: appInstance, + } + requestID := model.NewId() var statusCode string defer func() { @@ -157,6 +162,10 @@ func (h Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) { mlog.String("url", r.URL.Path), mlog.String("request_id", requestID), } + // if there is a session then include the user_id + if c.AppContext.Session() != nil { + responseLogFields = append(responseLogFields, mlog.String("user_id", c.AppContext.Session().UserId)) + } // Websockets are returning status code 0 to requests after closing the socket if statusCode != "0" { responseLogFields = append(responseLogFields, mlog.String("status_code", statusCode)) @@ -164,11 +173,6 @@ func (h Handler) ServeHTTP(w http.ResponseWriter, r *http.Request) { mlog.Debug("Received HTTP request", responseLogFields...) }() - c := &Context{ - AppContext: &request.Context{}, - App: appInstance, - } - t, _ := i18n.GetTranslationsAndLocaleFromRequest(r) c.AppContext = request.NewContext( context.Background(),