-----------------------------------------------------------
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
> 
>

Reply via email to