feat(api): use merge on conflict for last user application connection…

…s upsert
1024pix · Mar 4, 2025 · 1eee8a1 · 1eee8a1
1 parent 895a711
commit 1eee8a1
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/...ss-management/infrastructure/repositories/last-user-application-connections.repository.js b/...ss-management/infrastructure/repositories/last-user-application-connections.repository.js
@@ -2,13 +2,10 @@ import { knex } from '../../../../db/knex-database-connection.js';
 const TABLE_NAME = 'last-user-application-connections';
 
 async function upsert({ userId, application, lastLoggedAt }) {
-  const existingConnection = await knex(TABLE_NAME).where({ userId, application }).first();
-
-  if (existingConnection) {
-    return knex(TABLE_NAME).where({ userId, application }).update({ lastLoggedAt });
-  }
-
-  return knex(TABLE_NAME).insert({ userId, application, lastLoggedAt });
+  return knex(TABLE_NAME)
+    .insert({ userId, application, lastLoggedAt })
+    .onConflict(['userId', 'application'])
+    .merge({ lastLoggedAt });
 }
 
 export const lastUserApplicationConnectionsRepository = { upsert };