fix(rbac): fix uncommited knex transaction in the addGroupingPolicies (…

…janus-idp#1968) Signed-off-by: Oleksandr Andriienko <[email protected]>
debsmita1 · Jul 29, 2024 · 24d5eef · 24d5eef
1 parent a099514
commit 24d5eef
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/plugins/rbac-backend/src/service/enforcer-delegate.ts b/plugins/rbac-backend/src/service/enforcer-delegate.ts
@@ -79,12 +79,12 @@ export class EnforcerDelegate {
     policies: string[][],
     externalTrx?: Knex.Transaction,
   ): Promise<void> {
-    const trx = externalTrx || (await this.knex.transaction());
-
     if (policies.length === 0) {
       return;
     }
 
+    const trx = externalTrx || (await this.knex.transaction());
+
     try {
       const ok = await this.enforcer.addPolicies(policies);
       if (!ok) {