You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ranger.apache.org by Madhan Neethiraj <ma...@apache.org> on 2024/07/11 18:23:30 UTC

Re: Review Request 75050: RANGER-4777: Improve API /public/v2/api/service-headers to filter services depending on user role

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


Ship it!




Ship It!

- Madhan Neethiraj


On June 14, 2024, 2:43 p.m., Rakesh Gupta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/75050/
> -----------------------------------------------------------
> 
> (Updated June 14, 2024, 2:43 p.m.)
> 
> 
> Review request for ranger, Dineshkumar Yadav, Kishor Gollapalliwar, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, sanket shelar, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-4777
>     https://issues.apache.org/jira/browse/RANGER-4777
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Need to update the API - "/public/v2/api/service-headers" introduce in RANGER-4533 with below :
> 
> The API should be accessible for non-admin users as well. @PreAuthrize annotation can be removed.
> Filtering of services depending on user role like done for existing API - "/plugins/services"
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java 2aed957d8 
>   security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java 3aeda199a 
>   security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 883a22dd6 
>   security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIList.java 65e0f7848 
>   security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIMapping.java 59cd2a6dc 
> 
> 
> Diff: https://reviews.apache.org/r/75050/diff/1/
> 
> 
> Testing
> -------
> 
> Tested "/public/v2/api/service-headers" API is accessible for non-admin users.
> 
> 
> Thanks,
> 
> Rakesh Gupta
> 
>