-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73759/
-----------------------------------------------------------
(Updated Dec. 15, 2021, 8:28 a.m.)
Review request for ranger, Abhishek Kumar, Ankita Sinha, Don Bosco Durai,
Abhay Kulkarni, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and
Velmurugan Periasamy.
Changes
-------
updated to set default value for enableJsonCtx depending upon condition script
expression
Bugs: RANGER-3550
https://issues.apache.org/jira/browse/RANGER-3550
Repository: ranger
Description
-------
updated row-filter and condition handling to evaluate expressions given under
markers '${{' and '}}'
Diffs (updated)
-----
agents-common/src/main/java/org/apache/ranger/plugin/conditionevaluator/RangerScriptConditionEvaluator.java
b94225b36
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerRequestScriptEvaluator.java
64f01c95f
agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultRowFilterPolicyItemEvaluator.java
0831dde08
agents-common/src/main/java/org/apache/ranger/plugin/util/RangerCommonConstants.java
71029dd12
agents-common/src/main/java/org/apache/ranger/plugin/util/RangerRequestExprResolver.java
PRE-CREATION
agents-common/src/main/java/org/apache/ranger/plugin/util/ScriptEngineUtil.java
b0782ecb5
agents-common/src/test/java/org/apache/ranger/plugin/conditionevaluator/RangerCustomConditionMatcherTest.java
db7d3e582
agents-common/src/test/java/org/apache/ranger/plugin/conditionevaluator/RangerRequestScriptEvaluatorTest.java
798da052f
agents-common/src/test/java/org/apache/ranger/plugin/util/RangerRequestExprResolverTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/73759/diff/2/
Changes: https://reviews.apache.org/r/73759/diff/1-2/
Testing
-------
- added test cases to validate evaluation of expressions
- verified Hive plugin row-filter expressions that include user attribute,
like: "owner == '${{USER._name}}'
Thanks,
Madhan Neethiraj
