----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66589/#review201107 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On April 13, 2018, 3:16 p.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66589/ > ----------------------------------------------------------- > > (Updated April 13, 2018, 3:16 p.m.) > > > Review request for atlas, Ashutosh Mestry and Madhan Neethiraj. > > > 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 > ----- > > 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/3/ > > > 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 > >