-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74485/
-----------------------------------------------------------

(Updated July 12, 2023, 9:30 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 (updated)
-----

  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 a4dd6ef89 
  security-admin/src/main/java/org/apache/ranger/rest/SecurityZoneREST.java 
66fa22163 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java 
ce26b1d66 
  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
 d74f9a83e 
  
security-admin/src/main/webapp/react-webapp/src/views/PolicyListing/AddUpdatePolicyForm.jsx
 cc34488ea 
  
security-admin/src/main/webapp/react-webapp/src/views/UserGroupRoleListing/users_details/EditUserView.jsx
 e47fccc6f 


Diff: https://reviews.apache.org/r/74485/diff/2/

Changes: https://reviews.apache.org/r/74485/diff/1-2/


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

Reply via email to