Gracefull error messages on password mishap

middleware/authenticated.go

@@ -55,12 +55,14 @@ func Authenticated(app core.App) func(*core.RequestEvent) error {
 		if session.IsExpired() {
 			slog.Warn("Session expired", "user", user.Id, "name", user.Name, "session", session.ID)
 			cache.Delete(cookie.Value)
-			r, err := app.FindRecordById(dbmodels.SESSIONS_TABLE, session.ID)
-			e.SetCookie(deact_cookie)
-			e.Response.Header().Set("Clear-Site-Data", "\"cookies\"")
-			if err == nil {
-				app.Delete(r)
-			}
+			go func() {
+				r, err := app.FindRecordById(dbmodels.SESSIONS_TABLE, session.ID)
+				e.SetCookie(deact_cookie)
+				e.Response.Header().Set("Clear-Site-Data", "\"cookies\"")
+				if err == nil {
+					app.Delete(r)
+				}
+			}()
 			return e.Next()
 		}
 

pages/login.go

@@ -19,15 +19,17 @@ const (
 	TEMPLATE_LOGIN = "/login/"
 )
 
+var CSRF_CACHE *security.CSRFProtector
+
 // TODO:
 // - rate limiting
-// - maybe csrf
 
 func init() {
 	csrf_cache, err := security.NewCSRFProtector(time.Minute*5, time.Minute)
 	if err != nil {
 		panic(err)
 	}
+	CSRF_CACHE = csrf_cache
 
 	lp := &LoginPage{
 		StaticPage: pagemodels.StaticPage{
@@ -36,47 +38,60 @@ func init() {
 			Template: TEMPLATE_LOGIN,
 			URL:      URL_LOGIN,
 		},
-		csrf_cache: csrf_cache,
 	}
 	app.Register(lp)
 }
 
 type LoginPage struct {
 	pagemodels.StaticPage
-	csrf_cache *security.CSRFProtector
 }
 
 func (p *LoginPage) Setup(router *router.Router[*core.RequestEvent], app core.App, engine *templating.Engine) error {
-	router.GET(URL_LOGIN, p.GET(engine))
+	router.GET(URL_LOGIN, p.GET(engine, app))
 	router.POST(URL_LOGIN, p.POST(engine, app))
 	return nil
 }
 
-func (p *LoginPage) GET(engine *templating.Engine) HandleFunc {
+func (p *LoginPage) GET(engine *templating.Engine, app core.App) HandleFunc {
 	return func(e *core.RequestEvent) error {
 		data := make(map[string]any)
 		data["record"] = p
-		nonce, token, err := p.csrf_cache.GenerateTokenBundle()
+		nonce, token, err := CSRF_CACHE.GenerateTokenBundle()
 		if err != nil {
 			return engine.Response500(e, err, data)
 		}
 		data["csrf_nonce"] = nonce
 		data["csrf_token"] = token
 
-		// TODO: the function to delete tokens is not yet there
-		// as of right now, the tokens get only deleted from the clients
-		// We need to delete the tokens from the cache + table.
-		e.SetCookie(&http.Cookie{
-			Name:   dbmodels.SESSION_COOKIE_NAME,
-			Path:   "/",
-			MaxAge: -1,
-		})
-		e.Response.Header().Set("Clear-Site-Data", "\"cookies\"")
+		Logout(e, &app)
 
 		return engine.Response200(e, p.Template, data, p.Layout)
 	}
 }
 
+func Unauthorized(
+	engine *templating.Engine,
+	e *core.RequestEvent,
+	error error,
+	data map[string]any) error {
+
+	nonce, token, err := CSRF_CACHE.GenerateTokenBundle()
+	if err != nil {
+		return engine.Response500(e, err, data)
+	}
+
+	data["csrf_nonce"] = nonce
+	data["csrf_token"] = token
+	data["error"] = error.Error()
+
+	htm, err := engine.RenderToString(e, data, TEMPLATE_LOGIN, "blank")
+	if err != nil {
+		return engine.Response500(e, err, data)
+	}
+
+	return e.HTML(http.StatusUnauthorized, htm)
+}
+
 func (p *LoginPage) POST(engine *templating.Engine, app core.App) HandleFunc {
 	return func(e *core.RequestEvent) error {
 		data := make(map[string]any)
@@ -94,17 +109,19 @@ func (p *LoginPage) POST(engine *templating.Engine, app core.App) HandleFunc {
 			return engine.Response500(e, err, data)
 		}
 
-		if _, err := p.csrf_cache.ValidateTokenBundle(formdata.CsrfNonce, formdata.CsrfToken); err != nil {
-			return engine.Response403(e, err, data)
+		data["formdata"] = formdata
+
+		if _, err := CSRF_CACHE.ValidateTokenBundle(formdata.CsrfNonce, formdata.CsrfToken); err != nil {
+			return Unauthorized(engine, e, fmt.Errorf("Ungültiges CSRF-Token oder Zeit abgelaufen. Bitte versuchen Sie es erneut."), data)
 		}
 
 		if formdata.Username == "" || formdata.Password == "" {
-			return engine.Response403(e, fmt.Errorf("Username and password are required"), data)
+			return Unauthorized(engine, e, fmt.Errorf("Benuztername oder Passwort falsch. Bitte versuchen Sie es erneut."), data)
 		}
 
 		record, err := app.FindFirstRecordByData(dbmodels.USERS_TABLE, dbmodels.USERS_EMAIL_FIELD, formdata.Username)
 		if err != nil || !record.ValidatePassword(formdata.Password) {
-			return engine.Response403(e, err, data)
+			return Unauthorized(engine, e, fmt.Errorf("Benuztername oder Passwort falsch. Bitte versuchen Sie es erneut."), data)
 		}
 
 		duration := time.Minute * 60
@@ -138,10 +155,6 @@ func (p *LoginPage) POST(engine *templating.Engine, app core.App) HandleFunc {
 			})
 		}
 
-		redirect := "/reihen"
-		if r := e.Request.URL.Query().Get("redirectTo"); r != "" {
-			redirect = r
-		}
-		return e.Redirect(303, redirect)
+		return RedirectTo(e)
 	}
 }

pages/logout.go

@@ -28,25 +28,43 @@ type LogoutPage struct {
 }
 
 func (p *LogoutPage) Setup(router *router.Router[*core.RequestEvent], app core.App, engine *templating.Engine) error {
-	router.GET(p.URL, p.GET())
+	router.GET(p.URL, p.GET(app))
 	return nil
 }
 
-func (p *LogoutPage) GET() HandleFunc {
+func (p *LogoutPage) GET(app core.App) HandleFunc {
 	return func(e *core.RequestEvent) error {
-		// TODO: the function to delete tokens is not yet there
-		// as of right now, the tokens get only deleted from the clients
-		// We need to delete the tokens from the cache + table.
-		e.SetCookie(&http.Cookie{
-			Name:   dbmodels.SESSION_COOKIE_NAME,
-			Path:   "/",
-			MaxAge: -1,
-		})
-		e.Response.Header().Set("Clear-Site-Data", "\"cookies\"")
-		redirect := "/reihen"
-		if r := e.Request.URL.Query().Get("redirectTo"); r != "" {
-			redirect = r
-		}
-		return e.Redirect(303, redirect)
+		Logout(e, &app)
+		return RedirectTo(e)
 	}
 }
+
+func Logout(e *core.RequestEvent, app *core.App) {
+	e.SetCookie(&http.Cookie{
+		Name:   dbmodels.SESSION_COOKIE_NAME,
+		Path:   "/",
+		MaxAge: -1,
+	})
+
+	e.Response.Header().Set("Clear-Site-Data", "\"cookies\"")
+
+	cookie, err := e.Request.Cookie(dbmodels.SESSION_COOKIE_NAME)
+	if err == nil && app != nil {
+		go func() {
+			app := *app
+			record, err := app.FindFirstRecordByData(dbmodels.SESSIONS_TABLE, dbmodels.SESSIONS_TOKEN_FIELD, cookie.Value)
+			if err == nil && record != nil {
+				app.Delete(record)
+			}
+		}()
+	}
+}
+
+func RedirectTo(e *core.RequestEvent) error {
+	redirect := "/reihen"
+
+	if r := e.Request.URL.Query().Get("redirectTo"); r != "" {
+		redirect = r
+	}
+	return e.Redirect(303, redirect)
+}

templating/engine.go

@@ -252,25 +252,28 @@ func (e *Engine) Render(out io.Writer, path string, ld map[string]any, layout ..
 	return nil
 }
 
-func (e *Engine) Response403(request *core.RequestEvent, err error, data map[string]any) error {
-	if data == nil {
-		data = make(map[string]any)
+func (e *Engine) RenderToString(request *core.RequestEvent, ld map[string]any, path string, layout ...string) (string, error) {
+	if ld == nil {
+		ld = make(map[string]any)
 	}
 
-	var sb strings.Builder
+	ld["page"] = requestData(request)
+
+	var builder strings.Builder
+	err := e.Render(&builder, path, ld, layout...)
 	if err != nil {
-		request.App.Logger().Error("Unauthorized 403 error fetching URL!", "error", err, "request", request.Request.URL)
-		data["Error"] = err.Error()
+		return "", e.Response500(request, err, ld)
 	}
 
-	data["page"] = requestData(request)
-
-	err2 := e.Render(&sb, "/errors/403/", data)
-	if err2 != nil {
-		return e.Response500(request, errors.Join(err, err2), data)
+	tstring := builder.String()
+	if e.debug {
+		idx := strings.LastIndex(tstring, "</body>")
+		if idx != -1 {
+			tstring = tstring[:idx] + RELOAD_TEMPLATE + tstring[idx:]
+		}
 	}
 
-	return request.HTML(http.StatusNotFound, sb.String())
+	return tstring, nil
 }
 
 func (e *Engine) Response404(request *core.RequestEvent, err error, data map[string]any) error {

views/layouts/components/_footer.gohtml

@@ -20,8 +20,11 @@
 			<span>&middot;</span>
 			{{ if .page.User }}
 				<i class="ri-user-3-line"></i>
-				Eingeloggt als
-				{{ .page.User.Email }}
+				{{ if .page.User.Name }}
+					<b>{{ .page.User.Name }}</b>
+				{{ else }}
+					<b>{{ .page.User.Email }}</b>
+				{{ end }}
 				|
 				<i class="ri-logout-box-line"></i>
 				<a href="/logout?redirectTo={{ .page.FullPath }}">Logout</a>

views/routes/login/body.gohtml

@@ -11,8 +11,15 @@
 				<img class="h-20 w-20 border" src="/assets/favicon.png" />
 			</div>
 		</div>
-		<h1 class="text-4xl self-baseline text-center w-full mt-6">Musenalm | Login</h1>
-		<form class="mt-9 w-full grid grid-cols-3 gap-4" method="POST">
+		<h1 class="text-4xl self-baseline text-center w-full my-6">Musenalm | Login</h1>
+		{{ if $model.error }}
+			<div
+				class="text-red-800 text-sm mt-2 rounded bg-red-200 p-2 font-bold border-red-700
+				border-2 mb-3">
+				{{ $model.error }}
+			</div>
+		{{ end }}
+		<form class="w-full grid grid-cols-3 gap-4" method="POST">
 			<div
 				class="col-span-3 border-2 border-transparent focus-within:border-slate-600 px-2 py-1 pb-1.5
 				bg-slate-200 focus-within:bg-slate-50 rounded-md transition-all duration-100">