Skip to content

Commit bfadb3d

Browse files
committed
Adding better logging
1 parent 0d09f1a commit bfadb3d

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

views/helpers.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ func (v *Views) getSessionData(eC echo.Context) *Context {
112112
func (v *Views) setMessagesInSession(eC echo.Context, c *Context) error {
113113
session, err := v.cookie.Get(eC.Request(), v.conf.SessionCookieName)
114114
if err != nil {
115-
return fmt.Errorf("error getting session: %w", err)
115+
return fmt.Errorf("failed to get session for set message: %w", err)
116116
}
117117

118118
session.Values["internalContext"] = InternalContext{
@@ -132,7 +132,7 @@ func (v *Views) setMessagesInSession(eC echo.Context, c *Context) error {
132132
func (v *Views) clearMessagesInSession(eC echo.Context) error {
133133
session, err := v.cookie.Get(eC.Request(), v.conf.SessionCookieName)
134134
if err != nil {
135-
return fmt.Errorf("error getting session: %w", err)
135+
return fmt.Errorf("failed to get session for clear message: %w", err)
136136
}
137137

138138
session.Values["internalContext"] = InternalContext{}

views/middleware.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -24,17 +24,17 @@ func (v *Views) RequiresLogin(next echo.HandlerFunc) echo.HandlerFunc {
2424
return func(c echo.Context) error {
2525
session, err := v.cookie.Get(c.Request(), v.conf.SessionCookieName)
2626
if err != nil {
27-
log.Printf("failed to get session: %+v", err)
27+
log.Printf("failed to get session for requires login: %+v", err)
2828

2929
session, err = v.cookie.New(c.Request(), v.conf.SessionCookieName)
3030
if err != nil {
31-
log.Printf("failed to make new session: %+v", err)
31+
log.Printf("failed to make new session for requires login: %+v", err)
3232
return v.LogoutFunc(c)
3333
}
3434

3535
err = session.Save(c.Request(), c.Response())
3636
if err != nil {
37-
log.Printf("failed to save session for logout: %+v", err)
37+
log.Printf("failed to save session for requires login: %+v", err)
3838
}
3939

4040
return c.Redirect(http.StatusFound, "/")

views/user.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ func (v *Views) AssumeUserFunc(c echo.Context) error {
300300
if c.Request().Method == http.MethodPost {
301301
session, err := v.cookie.Get(c.Request(), v.conf.SessionCookieName)
302302
if err != nil {
303-
return fmt.Errorf("error getting session: %w", err)
303+
return fmt.Errorf("failed to get session for assume user: %w", err)
304304
}
305305

306306
c1 := v.getSessionData(c)
@@ -342,7 +342,7 @@ func (v *Views) ReleaseUserFunc(c echo.Context) error {
342342
if c.Request().Method == http.MethodPost {
343343
session, err := v.cookie.Get(c.Request(), v.conf.SessionCookieName)
344344
if err != nil {
345-
return fmt.Errorf("error getting session: %w", err)
345+
return fmt.Errorf("failed to get session for release user: %w", err)
346346
}
347347

348348
c1 := v.getSessionData(c)

0 commit comments

Comments
 (0)