----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74292/#review225397 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/RoleRefUpdater.java Lines 329 (patched) <https://reviews.apache.org/r/74292/#comment314028> why ignored the exception ? - Pradeep Agrawal On April 18, 2023, 7:28 a.m., Rakesh Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74292/ > ----------------------------------------------------------- > > (Updated April 18, 2023, 7:28 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. > > > Bugs: RANGER-4025 > https://issues.apache.org/jira/browse/RANGER-4025 > > > Repository: ranger > > > Description > ------- > > Provide API for Roles import/export. > > > Diffs > ----- > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerRole.java > 682bbd640 > security-admin/src/main/java/org/apache/ranger/biz/RoleDBStore.java > c19e3e1a1 > security-admin/src/main/java/org/apache/ranger/biz/RoleRefUpdater.java > 421b2312d > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > 562467e80 > security-admin/src/main/java/org/apache/ranger/rest/RoleREST.java 4f0edd2b0 > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 5542250aa > > security-admin/src/main/java/org/apache/ranger/view/RangerExportRoleList.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/view/RangerPolicyList.java > 4799b3f03 > security-admin/src/main/java/org/apache/ranger/view/RangerRoleList.java > adbe93db6 > security-admin/src/test/java/org/apache/ranger/rest/TestServiceREST.java > 9d17553a4 > > > Diff: https://reviews.apache.org/r/74292/diff/4/ > > > Testing > ------- > > Tested the below Rest API's for Roles import/export to make sure everything > works fine. > > RoleREST Rest API :GET /roles/roles/exportJson > RoleREST Rest API :GET > /roles/roles/exportJson?roleName={rolename},{rolename},.... > > RoleREST Rest API :GET /roles/roles/importRolesFromFile > RoleREST Rest API :GET /roles/roles/importRolesFromFile?updateIfExists=true > RoleREST Rest API :GET > /roles/roles/importRolesFromFile?createNonExistUserGroupRole=true > > > Thanks, > > Rakesh Gupta > >