From 6411f168e2a543fbb30a9da18b646dfb88145f74 Mon Sep 17 00:00:00 2001 From: Trong Huu Nguyen Date: Fri, 7 Jan 2022 17:04:06 +0100 Subject: [PATCH] refactor(handler/login): move user agent debug statement to field --- pkg/router/handler_login.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/pkg/router/handler_login.go b/pkg/router/handler_login.go index 85d582b..5f4d2be 100644 --- a/pkg/router/handler_login.go +++ b/pkg/router/handler_login.go @@ -55,8 +55,11 @@ func (h *Handler) Login(w http.ResponseWriter, r *http.Request) { func (h *Handler) getLoginCookie(r *http.Request) (*openid.LoginCookie, error) { loginCookieJson, err := h.getDecryptedCookie(r, LoginCookieName) if err != nil { - log.Warnf("failed to fetch login cookie; falling back to legacy cookie: %+v", err) - log.Debugf("debug: user-agent: %s", r.UserAgent()) + requestFields := map[string]interface{}{ + "userAgent": r.UserAgent(), + } + log.WithField("httpRequest", requestFields). + Warnf("failed to fetch login cookie: %+v; falling back to legacy cookie", err) loginCookieJson, err = h.getDecryptedCookie(r, LoginLegacyCookieName) if err != nil { return nil, err