From 8d80e9103f836284b73bc8d176c5ca63c31b0dfc Mon Sep 17 00:00:00 2001 From: jvoisin Date: Mon, 4 Mar 2024 00:32:16 +0100 Subject: Delay call of `view.New` after logging the user in There is no need to do extra work like creating a session and its associated view until the user has been properly identified and as many possibly-failing sql request have been successfully run. --- internal/ui/api_key_create.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'internal/ui/api_key_create.go') diff --git a/internal/ui/api_key_create.go b/internal/ui/api_key_create.go index 05665037..9b1376ac 100644 --- a/internal/ui/api_key_create.go +++ b/internal/ui/api_key_create.go @@ -14,15 +14,14 @@ import ( ) func (h *handler) showCreateAPIKeyPage(w http.ResponseWriter, r *http.Request) { - sess := session.New(h.store, request.SessionID(r)) - view := view.New(h.tpl, r, sess) - user, err := h.store.UserByID(request.UserID(r)) if err != nil { html.ServerError(w, r, err) return } + sess := session.New(h.store, request.SessionID(r)) + view := view.New(h.tpl, r, sess) view.Set("form", &form.APIKeyForm{}) view.Set("menu", "settings") view.Set("user", user) -- cgit v1.2.3