ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Velmurugan Periasamy <vperias...@hortonworks.com>
Subject Re: Review Request 69526: RANGER-2308: User role user should not able to access usersync audit report if it does not have permissions on the audit module.
Date Wed, 12 Dec 2018 17:10:51 GMT

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


Ship it!




Ship It!

- Velmurugan Periasamy


On Dec. 10, 2018, 6 a.m., Pradeep Agrawal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/69526/
> -----------------------------------------------------------
> 
> (Updated Dec. 10, 2018, 6 a.m.)
> 
> 
> Review request for ranger, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P,
Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2308
>     https://issues.apache.org/jira/browse/RANGER-2308
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Currently user is having default access to usersync audit report but it be should able
to access the report only when he is having access in audit module. without that user can't
see details in the UI which is not as per the default behaviour of dashboard for user role
users.
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/biz/AssetMgr.java 941691aaa 
>   security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java e1a6b5859 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 865e115d3 
>   security-admin/src/test/java/org/apache/ranger/biz/TestXUserMgr.java 471052f62 
>   security-admin/src/test/java/org/apache/ranger/rest/TestServiceREST.java a8e6e61a0

> 
> 
> Diff: https://reviews.apache.org/r/69526/diff/2/
> 
> 
> Testing
> -------
> 
> Tested at local with patch.
> 
> 
> Thanks,
> 
> Pradeep Agrawal
> 
>


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