ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Madhan Neethiraj <mad...@apache.org>
Subject Re: Review Request 44744: Scrub received policies before policy engine uses it to guard against inadvertant data corruption: remove null policy resource values
Date Sat, 12 Mar 2016 01:37:08 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44744/#review123250
-----------------------------------------------------------


Fix it, then Ship it!





agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
(line 471)
<https://reviews.apache.org/r/44744/#comment185459>

    "<==" -> "==>"



agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
(line 477)
<https://reviews.apache.org/r/44744/#comment185458>

    level ==> resourceName


- Madhan Neethiraj


On March 12, 2016, 1:26 a.m., Alok Lal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44744/
> -----------------------------------------------------------
> 
> (Updated March 12, 2016, 1:26 a.m.)
> 
> 
> Review request for ranger and Madhan Neethiraj.
> 
> 
> Bugs: RANGER-822
>     https://issues.apache.org/jira/browse/RANGER-822
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Scrub null values from policy resource before policy evaluators use them.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
e79b5cd 
> 
> Diff: https://reviews.apache.org/r/44744/diff/
> 
> 
> Testing
> -------
> 
> Reproduced the problem and tested the fix in an older version of the code.  Checked that
WARNING messsages shows up containing the policy id and resource level to identify the problem
source.
> 
> Apache JIRA seems to be down currently.  I'll update it with review request once I can
get at it.
> 
> 
> Thanks,
> 
> Alok Lal
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message