-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56694/
-----------------------------------------------------------
(Updated Feb. 17, 2017, 12:47 p.m.)
Review request for ranger, Don Bosco Durai, Gautam Borad, Pradeep Agrawal, and
Velmurugan Periasamy.
Changes
-------
Improvised on escaping the special characters to be shown in UI.
Bugs: RANGER-1380
https://issues.apache.org/jira/browse/RANGER-1380
Repository: ranger
Description
-------
Not able to delete group that is having special character from ranger admin.
Problem is the input sent for special character is appended with amp after the
& which is causing the issue.
Diffs (updated)
-----
security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 2373f35
security-admin/src/main/webapp/scripts/models/RangerServiceDef.js 9c3b585
security-admin/src/main/webapp/scripts/utils/XAUtils.js c68c596
security-admin/src/main/webapp/scripts/views/kms/KMSTableLayout.js 0a24bc1
security-admin/src/main/webapp/scripts/views/policies/PermissionList.js
b3a91d4
security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js
bdc2ef4
security-admin/src/main/webapp/scripts/views/service/ServiceCreate.js f348744
security-admin/src/main/webapp/scripts/views/users/GroupCreate.js a11dc47
security-admin/src/main/webapp/scripts/views/users/UserCreate.js a0af92d
security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js da39d39
security-admin/src/main/webapp/templates/helpers/XAHelpers.js 3559421
Diff: https://reviews.apache.org/r/56694/diff/
Testing
-------
Verified :
1) Import of a user with special character in username from LDAP / AD.
2) Import of a group with special character in groupname from LDAP / AD.
3) Delete of a user / group having & in username / groupname.
4) Listing and search of such users in Policy listing and edit policy page.
Thanks,
Mehul Parikh