----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/75237/ -----------------------------------------------------------
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 ----- agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceHeaderInfo.java 7303bc746 security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java 0d0102288 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 eb7fc05ae security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java 3aeda199a security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 435b78679 security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIList.java d8284dff6 security-admin/src/main/java/org/apache/ranger/security/context/RangerAPIMapping.java 59cd2a6dc security-admin/src/main/resources/META-INF/jpa_named_queries.xml 616ce9924 Diff: https://reviews.apache.org/r/75237/diff/1/ Testing ------- Tested "/public/v2/api/service-headers" API is accessible for non-admin users. Thanks, Rakesh Gupta