----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62382/#review186004 -----------------------------------------------------------
Ship it! Ship It! webapp/src/main/java/org/apache/atlas/web/filters/AtlasKnoxSSOAuthenticationFilter.java Lines 425 (patched) <https://reviews.apache.org/r/62382/#comment262352> You could remove this if you don't have corresponding tests. - Ashutosh Mestry On Sept. 22, 2017, 5:32 p.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62382/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2017, 5:32 p.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, keval bhatt, Madhan > Neethiraj, and Sarath Subramanian. > > > Bugs: ATLAS-2144 > https://issues.apache.org/jira/browse/ATLAS-2144 > > > Repository: atlas > > > Description > ------- > > This patch includes fix to add Knox x-forwarded path to Atlas base URL when > Atlas is access via knox proxy. > > Also the changed the httpSecurity filter precendence between SSOFilter and > basicFilter. > When request dispatched from knox proxy is landed in atlas with basic header > and hadoopJwt cookie header, the basicFilter is invoked first before > ssoFilter causing issue in SSO login. > > > Diffs > ----- > > > webapp/src/main/java/org/apache/atlas/web/filters/AtlasKnoxSSOAuthenticationFilter.java > 665fa34 > webapp/src/main/java/org/apache/atlas/web/security/AtlasSecurityConfig.java > 3bec838 > > > Diff: https://reviews.apache.org/r/62382/diff/2/ > > > Testing > ------- > > Tested Atlas with knox authentication > Tested Atlas knox proxy with Form login and Knox SSO. > Tested Atlas form based Login > Tested Atlas api with basic and knox cookie header. > > > Thanks, > > Nixon Rodrigues > >
