Skip to content

Commit bc46a10

Browse files
jerenmbGitHub Enterprise
authored and
GitHub Enterprise
committed
Test/casbin fix (#673)
1 parent 43f5037 commit bc46a10

File tree

1 file changed

+5
-5
lines changed
  • server/src/domain_objects/data_warehouse/ontology

1 file changed

+5
-5
lines changed

server/src/domain_objects/data_warehouse/ontology/container.ts

+5-5
Original file line numberDiff line numberDiff line change
@@ -305,17 +305,14 @@ export class ContainerPermissionSet extends NakedDomainClass {
305305
await e.removePolicy(userID, containerID, 'users', 'read');
306306
await e.removePolicy(userID, containerID, 'users', 'write');
307307

308-
try {
309-
await e.savePolicy();
310-
} catch (e) {
311-
Logger.debug(`error saving policy ${e}`);
312-
}
308+
Logger.info(`removed all casbin policies for user ID ${userID}`);
313309

314310
if (this.containers && this.containers.length > 0) {
315311
this.containers.forEach((permission) => {
316312
e.addPolicy(userID, containerID, 'containers', permission).catch((e) =>
317313
Logger.error(`error while setting container permissions for user ${userID}: ${e}`),
318314
);
315+
Logger.info(`added casbin policy container ${permission} for user ID ${userID}`);
319316
});
320317
}
321318

@@ -324,6 +321,7 @@ export class ContainerPermissionSet extends NakedDomainClass {
324321
e.addPolicy(userID, containerID, 'ontology', permission).catch((e) =>
325322
Logger.error(`error while setting container permissions for user ${userID}: ${e}`),
326323
);
324+
Logger.info(`added casbin policy ontology ${permission} for user ID ${userID}`);
327325
});
328326
}
329327

@@ -332,6 +330,7 @@ export class ContainerPermissionSet extends NakedDomainClass {
332330
e.addPolicy(userID, containerID, 'data', permission).catch((e) =>
333331
Logger.error(`error while setting container permissions for user ${userID}: ${e}`),
334332
);
333+
Logger.info(`added casbin policy data ${permission} for user ID ${userID}`);
335334
});
336335
}
337336

@@ -340,6 +339,7 @@ export class ContainerPermissionSet extends NakedDomainClass {
340339
e.addPolicy(userID, containerID, 'users', permission).catch((e) =>
341340
Logger.error(`error while setting container permissions for user ${userID}: ${e}`),
342341
);
342+
Logger.info(`added casbin policy users ${permission} for user ID ${userID}`);
343343
});
344344
}
345345

0 commit comments

Comments
 (0)