Skip to content

Commit

Permalink
Cleanup sessions at login
Browse files Browse the repository at this point in the history
  • Loading branch information
Piszmog committed May 9, 2024
1 parent 8e04fc5 commit 56a3bee
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 10 deletions.
5 changes: 5 additions & 0 deletions server/handler/signin.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,11 @@ func (h *Handler) Authenticate(w http.ResponseWriter, r *http.Request) {
SameSite: http.SameSiteStrictMode,
})

err = h.Database.Queries().DeleteOldUserSessions(r.Context(), user.ID)
if err != nil {
h.Logger.Warn("failed to delete old user sessions", "userID", user.ID, "error", err)
}

w.Header().Set("HX-Redirect", "/")
}

Expand Down
10 changes: 0 additions & 10 deletions server/middleware/auth.go
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
package middleware

import (
"context"
"database/sql"
"log/slog"
"net/http"
Expand Down Expand Up @@ -76,17 +75,8 @@ func (m *AuthMiddleware) Middleware(next http.Handler) http.Handler {
}
}

m.cleanupUserSessions(context.Background(), session.UserID)

r.Header.Set("USER-ID", strconv.FormatInt(session.UserID, 10))

next.ServeHTTP(w, r)
})
}

func (m *AuthMiddleware) cleanupUserSessions(ctx context.Context, userID int64) {
err := m.Database.Queries().DeleteOldUserSessions(ctx, userID)
if err != nil {
m.Logger.Warn("failed to delete old user sessions", "userID", userID, "error", err)
}
}

0 comments on commit 56a3bee

Please sign in to comment.