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

(Updated July 15, 2024, 1:51 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.


Changes
-------

Resolved merge conflict


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 (updated)
-----

  security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java 
546f511c2 
  security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java 
3aeda199a 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
be56c487f 
  
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/2/

Changes: https://reviews.apache.org/r/75050/diff/1-2/


Testing
-------

Tested "/public/v2/api/service-headers" API is accessible for non-admin users.


Thanks,

Rakesh Gupta

Reply via email to