----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74485/#review225584 -----------------------------------------------------------
Ship it! Ship It! - Mugdha Varadkar On June 21, 2023, 6:35 a.m., Brijesh Bhalala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74485/ > ----------------------------------------------------------- > > (Updated June 21, 2023, 6:35 a.m.) > > > Review request for ranger, Dhaval Rajpara, Dineshkumar Yadav, Madhan > Neethiraj, Mehul Parikh, Mugdha Varadkar, Nikunj Pansuriya, and Nitin Galave. > > > Bugs: RANGER-4132 > https://issues.apache.org/jira/browse/RANGER-4132 > > > Repository: ranger > > > Description > ------- > > In Ranger Admin UI, the policies list for a specific service are not updated > automatically. > So in case a page is open for a specific service and it contains a policy, > for e.g with policy id 99, and the policy is deleted from another tab / from > an API request, > and if the user clicks on the view policy button, the server returns a 400 > bad request error, but the page gets stuck with the message "Please wait". > In such scenarios, there should be a message which states that the policy has > been deleted and it should refresh the list of policies for a specific service > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java f653dbc8a > security-admin/src/main/java/org/apache/ranger/common/RESTErrorUtil.java > e0ff95814 > security-admin/src/main/java/org/apache/ranger/rest/RoleREST.java 7fb7d9c79 > security-admin/src/main/java/org/apache/ranger/rest/SecurityZoneREST.java > 66fa22163 > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > e91d3bc71 > > security-admin/src/main/webapp/react-webapp/src/components/CommonComponents.jsx > 7fe756c47 > security-admin/src/main/webapp/react-webapp/src/utils/fetchAPI.js 5d9a7d5c4 > > security-admin/src/main/webapp/react-webapp/src/views/AuditEvent/AdminLogs/PolicyLogs.jsx > abb58c94d > > security-admin/src/main/webapp/react-webapp/src/views/AuditEvent/AdminLogs/PolicyViewDetails.jsx > e5c8bc399 > > security-admin/src/main/webapp/react-webapp/src/views/PolicyListing/AddUpdatePolicyForm.jsx > a27cbaa28 > > security-admin/src/main/webapp/react-webapp/src/views/UserGroupRoleListing/users_details/EditUserView.jsx > e47fccc6f > > > Diff: https://reviews.apache.org/r/74485/diff/1/ > > > Testing > ------- > > 1)Build and Verified Ranger Admin setup with this changes. > 2)Verified the following things:- > - CRUD Operation for Policy module > - multiple tab scenarios for the policies, service & users/groups/roles > > > Thanks, > > Brijesh Bhalala > >