-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66589/
-----------------------------------------------------------

(Updated April 13, 2018, 1:31 p.m.)


Review request for atlas, Ashutosh Mestry and Madhan Neethiraj.


Changes
-------

This patch handles the review comments from Madhan N.


Bugs: ATLAS-2557
    https://issues.apache.org/jira/browse/ATLAS-2557


Repository: atlas


Description
-------

This patch includes fix to allow to lookup hadoop ldap groups when group from 
are wrongly set.
In this case reading groups from hadoop ldap group mapping can help to get all 
groups.


Diffs (updated)
-----

  intg/src/main/java/org/apache/atlas/utils/AuthenticationUtil.java 09d8085c1 
  
webapp/src/main/java/org/apache/atlas/web/security/AtlasAbstractAuthenticationProvider.java
 f77cb01d1 


Diff: https://reviews.apache.org/r/66589/diff/2/

Changes: https://reviews.apache.org/r/66589/diff/1-2/


Testing
-------

Tested with property atlas.authentication.ugi-groups.include-hadoop-groups set 
true and false. to verify the groups return are correct.


Thanks,

Nixon Rodrigues

Reply via email to