> On March 7, 2017, 1:35 p.m., Velmurugan Periasamy wrote: > > knox-agent/src/main/java/org/apache/ranger/authorization/knox/KnoxRangerPlugin.java > > Lines 93 (patched) > > <https://reviews.apache.org/r/57373/diff/1/?file=1658118#file1658118line93> > > > > Duplicate of RANGER-1422 ?
Thanks Vel, yes Knox assess type population issue part is covered in RANGER-1422 so updated patch with the email-id related issue. - Ankita ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57373/#review168124 ----------------------------------------------------------- On March 8, 2017, 4:19 a.m., Ankita Sinha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57373/ > ----------------------------------------------------------- > > (Updated March 8, 2017, 4:19 a.m.) > > > Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, > Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja > Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-1428 > https://issues.apache.org/jira/browse/RANGER-1428 > > > Repository: ranger > > > Description > ------- > > 1. Access type for knox is not displayed under Audit tab > 2. Junk email id is displayed in some scenario > > **Steps to reproduce point 2** > When added non-existing user while creating a service, it gets created in > user listing without email id but when same user is deleted it shows some > random data in email field in “User Profile deleted” entry > > Steps to follow: > 1. Create a service with non-existing user(user will get created in user > listing) > 2. Now delete that user > 3. Go to Audit-Admin tab and check log for "User Profile deleted", in > email-id field it will show some random data > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 62cffa5 > > > Diff: https://reviews.apache.org/r/57373/diff/2/ > > > Testing > ------- > > Tested on simple as well as secure cluster > > > Thanks, > > Ankita Sinha > >
