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

(Updated Oct. 16, 2023, 7:31 a.m.)


Review request for ranger, Brijesh Bhalala, Dhaval Shah, Dineshkumar Yadav, 
Kishor Gollapalliwar, Madhan Neethiraj, Mehul Parikh, Mugdha Varadkar, Pradeep 
Agrawal, and Velmurugan Periasamy.


Bugs: RANGER-4461
    https://issues.apache.org/jira/browse/RANGER-4461


Repository: ranger


Description
-------

Implement best coding practices for validating user input


Diffs
-----

  security-admin/src/main/webapp/scripts/modules/XAOverrides.js 369b0f65a 
  security-admin/src/main/webapp/scripts/utils/XAUtils.js 3c5907099 
  security-admin/src/main/webapp/scripts/views/kms/KMSTableLayout.js 011649200 
  
security-admin/src/main/webapp/scripts/views/permissions/ModulePermissionForm.js
 7b8c4c809 
  security-admin/src/main/webapp/scripts/views/policies/PermissionList.js 
ca3db854d 
  
security-admin/src/main/webapp/scripts/views/policies/RangerPolicyConditions.js 
d6c5295c1 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js 
118dfe215 
  security-admin/src/main/webapp/scripts/views/reports/OperationDiffDetail.js 
40cfd6d62 
  security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js 
ebdd2a0b2 
  security-admin/src/main/webapp/scripts/views/service/AuditFilterConfig.js 
27e2aaecb 
  security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js 
d443327c8 


Diff: https://reviews.apache.org/r/74673/diff/1/


Testing (updated)
-------

Validated below scenarios on old UI
1. Tested Resource Based/Tag Based/ KMS Service CRUD.
2. Tested Zone & Unzone policy CRUD.
3. Tested User/Group/ Role CRUD.
4. Tested Zone CRUD.
5. Tested Resource Based/Tag Based/ KMS Policy CRUD.
6. Tested reports/permissions/audits tab.
7. Tested Export and import feature.


Thanks,

Dhaval Rajpara

Reply via email to