From 40e4e4f4b8189f3c4e6f56a1f0b4835c8fde3309 Mon Sep 17 00:00:00 2001 From: Xevion Date: Sat, 9 Nov 2024 20:08:10 -0600 Subject: [PATCH] correctly add names to table constraints, fix inverted constraint for session_last_used_created --- .../linkpulse/migrations/006_add_session_constraints.py | 2 +- backend/linkpulse/models.py | 9 ++++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/backend/linkpulse/migrations/006_add_session_constraints.py b/backend/linkpulse/migrations/006_add_session_constraints.py index 1517f13..7e044de 100644 --- a/backend/linkpulse/migrations/006_add_session_constraints.py +++ b/backend/linkpulse/migrations/006_add_session_constraints.py @@ -48,7 +48,7 @@ def migrate(migrator: Migrator, database: pw.Database, *, fake=False): migrator.add_constraint( "session", "session_last_used_created_at", - pw.Check("last_used IS NULL OR last_used <= created_at"), + pw.Check("last_used IS NULL OR last_used >= created_at"), ) diff --git a/backend/linkpulse/models.py b/backend/linkpulse/models.py index c9fc192..dc6b640 100644 --- a/backend/linkpulse/models.py +++ b/backend/linkpulse/models.py @@ -70,9 +70,12 @@ class Session(BaseModel): class Meta: constraints = [ - Check("LENGTH(token) = 32"), - Check("expiry > created_at"), - Check("last_used IS NULL OR last_used <= created_at"), + Check("LENGTH(token) = 32", name="session_token_length"), + Check("expiry > created_at", name="session_expiry_created_at"), + Check( + "last_used IS NULL OR last_used >= created_at", + name="session_last_used_created_at", + ), ] @property