-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74572/
-----------------------------------------------------------
(Updated Nov. 20, 2023, 6:30 a.m.)
Review request for ranger, Dhaval Shah, Dineshkumar Yadav, Harshal Chavan,
Kishor Gollapalliwar, Madhan Neethiraj, Mehul Parikh, Mugdha Varadkar, Nitin
Galave, Pradeep Agrawal, and Velmurugan Periasamy.
Bugs: RANGER-4347
https://issues.apache.org/jira/browse/RANGER-4347
Repository: ranger
Description
-------
when using Ranger Audit filter to filter out the audit for a specific user:
Usually, if we input a username, it would be no issue, but if we input the user
like: “CN=SZH-C-004VX,CN=Machine,CN=PKI,DC=Bosch,DC=com”.
We found that once we saved the audit filter policy, the user was split into
“CN=SZH-C-004VX”, “CN=Machine”, “CN=PKI”, “DC=Bosch”, “DC=com”
It happens because Ranger UI recognize “,” to be separate items.
Same scenario is present in :
1) Security Zone Form
2) Permission modules
3) Role Form
Diffs
-----
security-admin/src/main/webapp/libs/fsOverrides/BBFOverrides.js 469f4aa9c
security-admin/src/main/webapp/scripts/models/RangerZone.js 08d118132
security-admin/src/main/webapp/scripts/utils/XAUtils.js 3c5907099
security-admin/src/main/webapp/scripts/views/permissions/ModulePermissionForm.js
7b8c4c809
security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js
ebdd2a0b2
security-admin/src/main/webapp/scripts/views/security_zone/ZoneCreate.js
a4885e379
security-admin/src/main/webapp/scripts/views/security_zone/ZoneCreateForm.js
cbfb8da2d
security-admin/src/main/webapp/scripts/views/service/AuditFilterConfig.js
27e2aaecb
security-admin/src/main/webapp/scripts/views/users/AddUsersOrGroupsList.js
26a62e67d
security-admin/src/main/webapp/styles/xa.css f5544cca0
security-admin/src/main/webapp/templates/permissions/ModulePermissionForm_tmpl.html
6aaaa8737
security-admin/src/main/webapp/templates/security_zone/ZoneCreateForm_tmpl.html
2ddfc3927
security-admin/src/main/webapp/templates/users/RoleForm_tmpl.html 7686c3fd1
Diff: https://reviews.apache.org/r/74572/diff/1/
Testing (updated)
-------
Validated below scenarios with adding comman in name 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
