----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71566/#review218018 -----------------------------------------------------------
Ship it! Ship It! - Zsombor Gegesy On Oct. 2, 2019, 12:33 p.m., Andor Molnar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71566/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2019, 12:33 p.m.) > > > Review request for ranger. > > > Repository: ranger > > > Description > ------- > > Currently RangerAuthorizationCoprocessor correctly captures all data from > HBase grant and revoke events, but RangerBasePlugin only copies certain > fields to RangerAccessEvent. > > RequestData is one the fields which are copied to the final entity and > currently not being used by the co-processor. I'd like to add some missing > information to this field and make it available on the UI similarly how Hive > queries are shown in a small pop-up. > > First, I change the co-processor to populate RequestData with additional > grant/revoke information. > > Second, I modify JS to show the RequestData pop-up on HBase audit events too. > > https://issues.apache.org/jira/browse/RANGER-2599 > > > Diffs > ----- > > > hbase-agent/src/main/java/org/apache/ranger/authorization/hbase/RangerAuthorizationCoprocessor.java > 364a415943bed22890dc5773e162962bb30147f9 > security-admin/src/main/webapp/scripts/utils/XAViewUtils.js > 6ef33435af7e7cbed7fbc5dc4174c8fb46be3aaf > > > Diff: https://reviews.apache.org/r/71566/diff/2/ > > > Testing > ------- > > Tested on private test cluster. > > > Thanks, > > Andor Molnar > >