----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74673/#review225881 -----------------------------------------------------------
Ship it! Ship It! - Mugdha Varadkar On Oct. 16, 2023, 7:31 a.m., Dhaval Rajpara wrote: > > ----------------------------------------------------------- > 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 > ------- > > 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 > >