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


Ship it!




Ship It!

- Abhay Kulkarni


On Nov. 5, 2024, 7:30 a.m., Mahesh Bandal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/75255/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2024, 7:30 a.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan 
> Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, 
> and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-4980
>     https://issues.apache.org/jira/browse/RANGER-4980
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> when chained plugin is enabled and all access is provided to a directory 
> which has multiple files and directories under it and if a user tries to 
> delete a that directory then access is denied.
> Steps to repro:
> create a policy which will grant all access to systest user on a directory.
> 
> Try performing write access. It should allow.
> * hdfs dfs -mkdir /user/data/part_default1/state=98/dir1
> 
> Now, remove default ACL's on hdfs path
> * hdfs dfs -chmod -R +000 /user/data/part_default1
> 
> Now, try delete operation. Access will be denied.
> * hdfs dfs -rm -r -skipTrash /user/data/part_default1/state=98
> 
> 
> Diffs
> -----
> 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java
>  8db08c598 
>   
> agents-common/src/main/java/org/apache/ranger/plugin/util/RangerAccessRequestUtil.java
>  6546e287c 
>   
> hdfs-agent/src/main/java/org/apache/ranger/authorization/hadoop/RangerHdfsAuthorizer.java
>  9ce25695c 
> 
> 
> Diff: https://reviews.apache.org/r/75255/diff/1/
> 
> 
> Testing
> -------
> 
> - manually tested the scenario.
> - ranger hdfs policy related sainty testing is done
> 
> 
> Thanks,
> 
> Mahesh Bandal
> 
>

Reply via email to