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 2021/08/03 18:31:58 UTC

[ranger] branch ranger-2.2 updated: RANGER-3360: Best Practice: Use updated policy object after pruning the policy object

This is an automated email from the ASF dual-hosted git repository.

abhay pushed a commit to branch ranger-2.2
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/ranger-2.2 by this push:
     new 5a07338  RANGER-3360: Best Practice: Use updated policy object after pruning the policy object
5a07338 is described below

commit 5a07338439dc9a1de10a3066ebe17aed7de2239c
Author: Abhay Kulkarni <ab...@apache.org>
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();