fix(rbac): fix defaultRoleManager hasRole method (#211)

alexey-yunoshev · web-flow · commit 4f3ba65429f9 · 2020-11-15T10:33:43.000+08:00
In the current version, defaultRoleManager hasRole always returns true if a matchingFunc is provided. This PR fixes it.

Signed-off-by: Alexey Yunoshev &lt;alexey.yunoshev@gmail.com&gt;
diff --git a/src/rbac/defaultRoleManager.ts b/src/rbac/defaultRoleManager.ts
@@ -99,7 +99,7 @@ class Roles extends Map<string, Role> {
     } else {
       return this.has(name);
     }
-    return true;
+    return ok;
   }
 
   public createRole(name: string, matchingFunc?: MatchingFunc): Role {

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ class Roles extends Map<string, Role> {`
`99`	`99`	`} else {`
`100`	`100`	`return this.has(name);`
`101`	`101`	`}`
`102`		`- return true;`
	`102`	`+ return ok;`
`103`	`103`	`}`
`104`	`104`
`105`	`105`	`public createRole(name: string, matchingFunc?: MatchingFunc): Role {`