This is an automated email from the ASF dual-hosted git repository.
abhay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git
The following commit(s) were added to refs/heads/master by this push:
new c9003ff RANGER-3360: Best Practice: Use updated policy object after
pruning the policy object
c9003ff is described below
commit c9003ff68a35bb1fa56d00b3cb2505ac00fbeb2e
Author: Abhay Kulkarni <[email protected]>
AuthorDate: Tue Aug 3 10:44:01 2021 -0700
RANGER-3360: Best Practice: Use updated policy object after pruning the
policy object
---
.../ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java | 2 +-
.../ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java | 2 ++
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git
a/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java
b/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java
index 5c6083e..52a30a1 100644
---
a/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java
+++
b/agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAbstractPolicyEvaluator.java
@@ -59,7 +59,7 @@ public abstract class RangerAbstractPolicyEvaluator
implements RangerPolicyEvalu
this.leafResourceDef =
ServiceDefUtil.getLeafResourceDef(serviceDef, getPolicyResource());
if(LOG.isDebugEnabled()) {
- LOG.debug("<== RangerAbstractPolicyEvaluator.init(" +
policy + ", " + serviceDef + ")");
+ LOG.debug("<== RangerAbstractPolicyEvaluator.init(" +
this.policy + ", " + serviceDef + ")");
}
}
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 831b6d4..b5b859c 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
@@ -123,6 +123,8 @@ public class RangerDefaultPolicyEvaluator extends
RangerAbstractPolicyEvaluator
super.init(policy, serviceDef, options);
+ policy = getPolicy();
+
preprocessPolicy(policy, serviceDef);
resourceMatcher = new RangerDefaultPolicyResourceMatcher();