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();

Reply via email to