fix: handle concurrent service account updates (#5349) (#5350)

Unleash · Nov 16, 2023 · 510fa30 · 510fa30
1 parent 4ca8456
commit 510fa30
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/src/lib/db/access-store.ts b/src/lib/db/access-store.ts
@@ -468,11 +468,14 @@ export class AccessStore implements IAccessStore {
         roleId: number,
         projectId?: string,
     ): Promise<void> {
-        return this.db(T.ROLE_USER).insert({
-            user_id: userId,
-            role_id: roleId,
-            project: projectId,
-        });
+        await this.db(T.ROLE_USER)
+            .insert({
+                user_id: userId,
+                role_id: roleId,
+                project: projectId,
+            })
+            .onConflict(['user_id', 'role_id', 'project'])
+            .ignore();
     }
 
     async removeUserFromRole(