----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72251/#review220012 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/SessionMgr.java Lines 184 (patched) <https://reviews.apache.org/r/72251/#comment308274> Please consider doing this check before line 182. If the check fails, then the rest of the condition(s) may be checked. - Abhay Kulkarni On March 19, 2020, 2:25 p.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72251/ > ----------------------------------------------------------- > > (Updated March 19, 2020, 2:25 p.m.) > > > Review request for ranger, Abhay Kulkarni, Mehul Parikh, Nitin Galave, Ramesh > Mani, and Velmurugan Periasamy. > > > Bugs: RANGER-2762 > https://issues.apache.org/jira/browse/RANGER-2762 > > > Repository: ranger > > > Description > ------- > > Made code changes to check if the browser requests are coming from knox when > trusted proxy is enabled before applying proper authentication filters. Also > labeled the browser session as "sso enabled" when session is created for knox > trusted proxy case. This label is used for handling logout from ranger UI and > redirect to appropriate page. > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/SessionMgr.java > ce09c36eb > > security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKrbFilter.java > b7b2b2ab3 > > security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerSecurityContextFormationFilter.java > 99fb21f59 > > > Diff: https://reviews.apache.org/r/72251/diff/1/ > > > Testing > ------- > > 1. Tested browser requests with and without knox trusted proxy enabled > 2. Also verified few regression cases. > > > Thanks, > > Sailaja Polavarapu > >
