-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66024/
-----------------------------------------------------------
(Updated March 15, 2018, 8:12 a.m.)
Review request for ranger.
Summary (updated)
-----------------
RANGER-2016 - fix the code cleanup
Bugs: RANGER-2016
https://issues.apache.org/jira/browse/RANGER-2016
Repository: ranger
Description (updated)
-------
There is a code path, where a null is passed as a HttpServletRequest - to
trigger 'searching with empty filter'.
A simple fix is to introduce a getServicePolicies(serviceName,SearchFilter)
method on ServiceREST class, so the HttpServletRequest is not needed anymore,
and this method can be called freely.
Diffs (updated)
-----
security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java
86b4e4309
security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java
229863e74
Diff: https://reviews.apache.org/r/66024/diff/2/
Changes: https://reviews.apache.org/r/66024/diff/1-2/
Testing
-------
Tested locally
Thanks,
Zsombor Gegesy
