-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74753/#review226062
-----------------------------------------------------------




security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java
Line 111 (original), 106 (patched)
<https://reviews.apache.org/r/74753/#comment314379>

    Please handle scenario when session is null



security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java
Lines 154 (patched)
<https://reviews.apache.org/r/74753/#comment314380>

    What will happen if session is null ?


- Kishor Gollapalliwar


On Dec. 14, 2023, 7:26 a.m., Rakesh Gupta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74753/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2023, 7:26 a.m.)
> 
> 
> Review request for ranger, Dineshkumar Yadav, Kishor Gollapalliwar, Abhay 
> Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, 
> sanket shelar, Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-4521
>     https://issues.apache.org/jira/browse/RANGER-4521
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Some audit entries for keyadmin user are missing when filtered using 
> sessionID or Audit Type = Ranger Policy. Also when moved to next page most 
> audit entries are missing
> 
> (Audits for import and export are shown, audit corresponding to policy crud 
> operation is missing for keyadmin user)
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/service/XTrxLogService.java 
> 9d9e0bbc0 
> 
> 
> Diff: https://reviews.apache.org/r/74753/diff/4/
> 
> 
> Testing
> -------
> 
> Tested All audits are displayed properly with filter and without filter for 
> admin and keyadmin user.
> 
> 
> Thanks,
> 
> Rakesh Gupta
> 
>

Reply via email to