-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/75050/
-----------------------------------------------------------
(Updated Oct. 18, 2024, 7:06 a.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
-------
Addressed Review Comments
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)
-----
agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceHeaderInfo.java
accf28721
security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java
2ece2e22a
security-admin/src/main/java/org/apache/ranger/db/XXSecurityZoneRefServiceDao.java
a7726d780
security-admin/src/main/java/org/apache/ranger/db/XXSecurityZoneRefTagServiceDao.java
9e1fb13ef
security-admin/src/main/java/org/apache/ranger/db/XXServiceDao.java 0a0144094
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
892414d8d
security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIMapping.java
59cd2a6dc
security-admin/src/main/resources/META-INF/jpa_named_queries.xml 47bb6e9af
Diff: https://reviews.apache.org/r/75050/diff/5/
Changes: https://reviews.apache.org/r/75050/diff/4-5/
Testing
-------
Tested "/public/v2/api/service-headers" API is accessible for non-admin users.
Thanks,
Rakesh Gupta
