fix: getImplicitPermissionsForUser missing domain parameter

nodece · nodece · commit 584624c99eab · 2020-01-14T23:09:00.000+08:00
diff --git a/src/enforcer.ts b/src/enforcer.ts
@@ -302,11 +302,16 @@ export class Enforcer extends ManagementEnforcer {
    * getPermissionsForUser("alice") can only get: [["alice", "data2", "read"]].
    * But getImplicitPermissionsForUser("alice") will get: [["admin", "data1", "read"], ["alice", "data2", "read"]].
    */
-  public async getImplicitPermissionsForUser(user: string): Promise<string[][]> {
-    const roles = [user, ...(await this.getImplicitRolesForUser(user))];
+  public async getImplicitPermissionsForUser(user: string, ...domain: string[]): Promise<string[][]> {
+    const roles = [user, ...(await this.getImplicitRolesForUser(user, ...domain))];
     const res: string[][] = [];
+    const withDomain = domain && domain.length !== 0;
     roles.forEach(n => {
-      res.push(...this.getPermissionsForUser(n));
+      if (withDomain) {
+        res.push(...this.getFilteredPolicy(0, n, ...domain));
+      } else {
+        res.push(...this.getPermissionsForUser(n));
+      }
     });
     return res;
   }
