You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ma...@apache.org on 2024/09/10 16:43:58 UTC
(ranger) 01/04: RANGER-4385: Fix role filtering logic in RolePredicateUtil (#280)
This is an automated email from the ASF dual-hosted git repository.
madhan pushed a commit to branch ranger-2.6
in repository https://gitbox.apache.org/repos/asf/ranger.git
commit f7c74f86f6607d18c521889123aa05163380488d
Author: bing <36...@qq.com>
AuthorDate: Tue Sep 10 02:16:09 2024 +0800
RANGER-4385: Fix role filtering logic in RolePredicateUtil (#280)
(cherry picked from commit a463e2fb7cdc70629142bd9808f2a84c03689a99)
---
.../main/java/org/apache/ranger/plugin/store/RolePredicateUtil.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/store/RolePredicateUtil.java b/agents-common/src/main/java/org/apache/ranger/plugin/store/RolePredicateUtil.java
index cda8d0999..48254987e 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/store/RolePredicateUtil.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/store/RolePredicateUtil.java
@@ -67,7 +67,7 @@ public class RolePredicateUtil extends AbstractPredicateUtil {
List<RangerRole.RoleMember> roles = role.getRoles();
for (RangerRole.RoleMember member : roles) {
- ret = StringUtils.equals(role.getName(), roleName);
+ ret = StringUtils.equals(member.getName(), roleName);
if (ret) {
break;
@@ -109,7 +109,7 @@ public class RolePredicateUtil extends AbstractPredicateUtil {
List<RangerRole.RoleMember> roles = role.getRoles();
for (RangerRole.RoleMember member : roles) {
- ret = StringUtils.containsIgnoreCase(role.getName(), roleNamePartial);
+ ret = StringUtils.containsIgnoreCase(member.getName(), roleNamePartial);
if (ret) {
break;