From 107a92f57304ab024b16898aae0345f9462f75dd Mon Sep 17 00:00:00 2001 From: kolaente Date: Wed, 25 Feb 2026 10:48:21 +0100 Subject: [PATCH] fix: commit transaction in session cleanup cron RegisterSessionCleanupCron opens a transaction via db.NewSession() but never calls s.Commit(). The deferred s.Close() auto-rolls-back, making the DELETE a no-op. Add the missing commit. --- pkg/models/sessions.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/pkg/models/sessions.go b/pkg/models/sessions.go index f63476648..06914279c 100644 --- a/pkg/models/sessions.go +++ b/pkg/models/sessions.go @@ -230,6 +230,10 @@ func RegisterSessionCleanupCron() { if deleted > 0 { log.Debugf(logPrefix+"Deleted %d stale sessions", deleted) } + + if err := s.Commit(); err != nil { + log.Errorf(logPrefix+"Could not commit: %s", err) + } }) if err != nil { log.Fatalf("Could not register session cleanup cron: %s", err)