----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74743/#review225968 -----------------------------------------------------------
Ship it! Ship It! - Selvamohan Neethiraj On Nov. 17, 2023, 3:36 p.m., Madhan Neethiraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74743/ > ----------------------------------------------------------- > > (Updated Nov. 17, 2023, 3:36 p.m.) > > > Review request for ranger, Anand Nadar, Ankita Sinha, Abhay Kulkarni, Mehul > Parikh, Ramesh Mani, Subhrat Chaudhary, and Velmurugan Periasamy. > > > Bugs: RANGER-4533 > https://issues.apache.org/jira/browse/RANGER-4533 > > > Repository: ranger > > > Description > ------- > > - REST API /public/v2/api/service-headers: added, with > optional query parameters serviceNamePrefix and serviceType > - REST API /public/v2/api/zone-headers: updated to > support optional query parameter zoneNamePrefix > - REST API /api/zones/zone-headers/for-service/{serviceId}: updated to > support optional query parameter zoneNamePrefix > - REST API /api/zones/{zoneId}/service-headers: updated to > support optional query parameter serviceNamePrefix > - RangerServiceHeaderInfo updated with fields: type, displayName - this would > improve the landing page by retrieving only service header info > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerServiceHeaderInfo.java > b9ea28b43 > agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java > 61f879894 > security-admin/src/main/java/org/apache/ranger/biz/SecurityZoneDBStore.java > 2228b89c5 > security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java > 62ff8e135 > > security-admin/src/main/java/org/apache/ranger/db/XXSecurityZoneRefServiceDao.java > c6a801191 > > security-admin/src/main/java/org/apache/ranger/db/XXSecurityZoneRefTagServiceDao.java > 1eaf0dec3 > security-admin/src/main/java/org/apache/ranger/db/XXServiceDao.java > ba92c7340 > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > eebab8108 > security-admin/src/main/java/org/apache/ranger/rest/SecurityZoneREST.java > 6513ad6b5 > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 5aee2a2c0 > security-admin/src/main/resources/META-INF/jpa_named_queries.xml 33172ce85 > security-admin/src/test/java/org/apache/ranger/rest/TestPublicAPIsv2.java > 73a593e9f > > > Diff: https://reviews.apache.org/r/74743/diff/2/ > > > Testing > ------- > > - verified in docker setup that REST APIs /public/v2/api/service-headers and > /public/v2/api/zone-headers returns service and zone headers respectively per > specified parameters > - verified that all existing unit tests pass successfully > > > Thanks, > > Madhan Neethiraj > >