> On Sept. 21, 2021, 7:02 p.m., Sailaja Polavarapu wrote: > > webapp/src/main/java/org/apache/atlas/web/filters/AtlasAuthenticationFilter.java > > Line 359 (original), 374 (patched) > > <https://reviews.apache.org/r/73599/diff/1/?file=2253162#file2253162line377> > > > > I think it will be intuitive to move logoutHandler != null check to > > inner if condition on line #375.
Please see my comment below. - Ashutosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73599/#review223509 ----------------------------------------------------------- On Sept. 22, 2021, 5:42 p.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73599/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2021, 5:42 p.m.) > > > Review request for atlas, Nikhil Bonte, Nixon Rodrigues, Prasad Pawar, and > Sarath Subramanian. > > > Bugs: ATLAS-4435 > https://issues.apache.org/jira/browse/ATLAS-4435 > > > Repository: atlas > > > Description > ------- > > Please see JIRA for details. > > Additional info: > * UI pop-up dialog looks at the value passed in the response of > 'admin/session' REST endpoint. Not passing the configuration value, disables > the feature from UI (Smart!). > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/AtlasConfiguration.java fa519ef5a > > webapp/src/main/java/org/apache/atlas/web/filters/AtlasAuthenticationFilter.java > b8d21b9bb > webapp/src/main/java/org/apache/atlas/web/resources/AdminResource.java > baa040f4c > > webapp/src/main/java/org/apache/atlas/web/security/AtlasAuthenticationSuccessHandler.java > 1b1a80826 > > > Diff: https://reviews.apache.org/r/73599/diff/2/ > > > Testing > ------- > > Manual testing: > * Verified against Knox SSO enabled cluster. > * User-name password enabled cluster. > > > Thanks, > > Ashutosh Mestry > >