----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48550/#review137235 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On June 10, 2016, 1:10 p.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48550/ > ----------------------------------------------------------- > > (Updated June 10, 2016, 1:10 p.m.) > > > Review request for atlas, Gautam Borad, Madhan Neethiraj, Mehul Parikh, > Shwetha GS, Suma Shivaprasad, and Hemanth Yamijala. > > > Bugs: ATLAS-869 > https://issues.apache.org/jira/browse/ATLAS-869 > > > Repository: atlas > > > Description > ------- > > Currently the LDAP/AD configartion properties are in atlas-admin-site.xml, > which is bundled with webapp. > End user should be able to configure it, it has been moved to > atlas-application.properties now. > > > Diffs > ----- > > distro/src/conf/atlas-application.properties bdd02d6 > > webapp/src/main/java/org/apache/atlas/web/security/AtlasADAuthenticationProvider.java > 34063b0 > > webapp/src/main/java/org/apache/atlas/web/security/AtlasLdapAuthenticationProvider.java > 6037858 > webapp/src/main/resources/atlas-admin-site.xml c4b4be7 > webapp/src/main/webapp/WEB-INF/applicationContext.xml 004c3a8 > > Diff: https://reviews.apache.org/r/48550/diff/ > > > Testing > ------- > > Tested login via FILE/LDAP/AD authentication methods. > > > Thanks, > > Nixon Rodrigues > >