----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74601/#review225783 -----------------------------------------------------------
Ship it! Ship It! - Mugdha Varadkar On Sept. 26, 2023, 1:26 p.m., Dineshkumar Yadav wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74601/ > ----------------------------------------------------------- > > (Updated Sept. 26, 2023, 1:26 p.m.) > > > Review request for ranger, Ankita Sinha, Dhaval Rajpara, Kishor > Gollapalliwar, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Mugdha > Varadkar, Pradeep Agrawal, Ramesh Mani, and Velmurugan Periasamy. > > > Bugs: RANGER-4399 > https://issues.apache.org/jira/browse/RANGER-4399 > > > Repository: ranger > > > Description > ------- > > This patch added open API for SecurityZoneHeaderInfo based on serviceId > service/public/v2/api/zones/zone-headers/for-service/{serviceId=}?isTagService=false > > where isTagService is false by default > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerSecurityZoneHeaderInfo.java > 02927b07b > security-admin/src/main/java/org/apache/ranger/biz/SecurityZoneDBStore.java > 44bca7489 > security-admin/src/main/java/org/apache/ranger/db/XXSecurityZoneDao.java > 5f73b64ea > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > 92ade823b > security-admin/src/main/java/org/apache/ranger/rest/SecurityZoneREST.java > 7c1e01053 > security-admin/src/main/resources/META-INF/jpa_named_queries.xml 23085a273 > security-admin/src/main/webapp/react-webapp/src/views/Home.jsx ffbc990fa > > security-admin/src/main/webapp/react-webapp/src/views/PolicyListing/PolicyListingTabView.jsx > 633290f75 > > security-admin/src/main/webapp/react-webapp/src/views/ServiceManager/ImportPolicy.jsx > 44346bb4b > > security-admin/src/main/webapp/react-webapp/src/views/ServiceManager/ServiceDefinitions.jsx > aeb88161b > security-admin/src/main/webapp/react-webapp/src/views/SideBar/TopNavBar.jsx > 4f3dedde1 > > > Diff: https://reviews.apache.org/r/74601/diff/3/ > > > Testing > ------- > > Tested UI patch manually. > > Tested manualy using curl request > -- for non-tag based service, below curl works > curl -u {user}:{user_pass} -X GET > '{hostname}/service/public/v2/api/zones/zone-headers/for-service/{serviceId}?isTagService=false' > curl -u {user}:{user_pass} -X GET > '{hostname}/service/public/v2/api/zones/zone-headers/for-service/{serviceId}' > > -- for tag based service need to pass isTagService=ture > curl -u {user}:{user_pass} -X GET > '{hostname}/service/public/v2/api/zones/zone-headers/for-service/{serviceId}?isTagService=true' > > > Thanks, > > Dineshkumar Yadav > >
