----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43426/#review119916 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On Feb. 17, 2016, 6:26 p.m., Sailaja Polavarapu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43426/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2016, 6:26 p.m.) > > > Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan > Neethiraj, Ramesh Mani, and Velmurugan Periasamy. > > > Bugs: Ranger-803 > https://issues.apache.org/jira/browse/Ranger-803 > > > Repository: ranger > > > Description > ------- > > Made code changes to add support for Multiple OUs in Ldap search for > usersync. Also added support to configure multiple DN patterns for ranger > admin authentication in order to support authentication of the users in > different OUs. Updated in-memory ldap schema and added more unit tests to > test multiple OUs changes for ranger usersync. > > > Diffs > ----- > > > security-admin/src/main/java/org/apache/ranger/security/handler/RangerAuthenticationProvider.java > cfdd9bc > > ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java > 529af72 > > ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java > 43cd982 > ugsync/src/test/java/org/apache/ranger/usergroupsync/LdapUserGroupTest.java > ae87aee > ugsync/src/test/resources/ADSchema.ldif 092d018 > > Diff: https://reviews.apache.org/r/43426/diff/ > > > Testing > ------- > > 1. Ran updated unit tests to test the recent changes. > 2. Tested functionality of usersync against OpenLdap Server with multiple OUs > for both users & groups. > 3. Tested functionality of ranger admin authentication with users from two > different OUs. > 4. Also performed some regression tests with basic configuration. > > > Thanks, > > Sailaja Polavarapu > >
