use replace strategy to trigger proper session cleanup

nais · Jul 8, 2024 · dace180 · dace180
1 parent ad91239
commit dace180
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/internal/apiserver/database/queries/sessions.sql b/internal/apiserver/database/queries/sessions.sql
@@ -15,12 +15,9 @@ ORDER BY s.expiry DESC
 LIMIT 1;
 
 -- name: AddSession :exec
-INSERT INTO sessions (key, expiry, device_id, object_id)
-VALUES (@key, @expiry, @device_id, @object_id)
-ON CONFLICT (device_id) DO UPDATE
-SET key = excluded.key,
-    expiry = excluded.expiry,
-    object_id = excluded.object_id;
+INSERT OR REPLACE INTO sessions (key, expiry, device_id, object_id)
+VALUES (@key, @expiry, @device_id, @object_id);
+
 
 -- name: AddSessionAccessGroupID :exec
 INSERT INTO session_access_group_ids (session_key, group_id)

diff --git a/internal/apiserver/sqlc/sessions.sql.go b/internal/apiserver/sqlc/sessions.sql.go