You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ab...@apache.org on 2018/08/23 22:06:15 UTC

ranger git commit: RANGER-2197: Delegate Admin is not able to create policy

Repository: ranger
Updated Branches:
  refs/heads/master 12aebc9e6 -> 41edc4675


RANGER-2197: Delegate Admin is not able to create policy


Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/41edc467
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/41edc467
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/41edc467

Branch: refs/heads/master
Commit: 41edc467584032e4c89f03aae86824ec28a7c154
Parents: 12aebc9
Author: Abhay Kulkarni <ak...@hortonworks.com>
Authored: Thu Aug 23 14:59:19 2018 -0700
Committer: Abhay Kulkarni <ak...@hortonworks.com>
Committed: Thu Aug 23 14:59:19 2018 -0700

----------------------------------------------------------------------
 .../plugin/policyevaluator/RangerDefaultPolicyEvaluator.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ranger/blob/41edc467/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
----------------------------------------------------------------------
diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java b/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
index 76ff0f7..289ec9b 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java
@@ -740,7 +740,7 @@ public class RangerDefaultPolicyEvaluator extends RangerAbstractPolicyEvaluator
 			LOG.debug("==> RangerDefaultPolicyEvaluator.isMatch(" + policy.getId() + ", " + evalContext + ")");
 		}
 
-		boolean ret = policy.getId() == getId() || isMatch(policy.getResources(), evalContext);
+		final boolean ret = isMatch(policy.getResources(), evalContext);
 
 		if(LOG.isDebugEnabled()) {
 			LOG.debug("<== RangerDefaultPolicyEvaluator.isMatch(" + policy.getId() + ", " + evalContext + "): " + ret);