-----------------------------------------------------------
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
> 
>

Reply via email to