You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Velmurugan Periasamy <vp...@hortonworks.com> on 2016/02/19 19:40:59 UTC

Re: Review Request 43426: Ranger-803: Support multiple OU in LDAP search for Ranger usersync

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