You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by sp...@apache.org on 2017/03/10 00:24:22 UTC
ranger git commit: RANGER-1434: Enable Group Search First causes
issues when Enable Group Sync is disabled - ranger-0.7 branch
Repository: ranger
Updated Branches:
refs/heads/ranger-0.7 6d96b21a3 -> 6ec7d9659
RANGER-1434: Enable Group Search First causes issues when Enable Group Sync is disabled - ranger-0.7 branch
Project: http://git-wip-us.apache.org/repos/asf/ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/6ec7d965
Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/6ec7d965
Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/6ec7d965
Branch: refs/heads/ranger-0.7
Commit: 6ec7d9659d26925795ecb65bd5e3c8a89982116a
Parents: 6d96b21
Author: Sailaja Polavarapu <sp...@hortonworks.com>
Authored: Thu Mar 9 16:24:14 2017 -0800
Committer: Sailaja Polavarapu <sp...@hortonworks.com>
Committed: Thu Mar 9 16:24:14 2017 -0800
----------------------------------------------------------------------
.../unixusersync/config/UserGroupSyncConfig.java | 3 +++
.../apache/ranger/usergroupsync/TestLdapUserGroup.java | 13 ++++++++++++-
2 files changed, 15 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ranger/blob/6ec7d965/ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
----------------------------------------------------------------------
diff --git a/ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java b/ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
index d839b39..81ddaa9 100644
--- a/ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
+++ b/ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java
@@ -750,6 +750,9 @@ public class UserGroupSyncConfig {
} else {
groupSearchFirstEnabled = Boolean.valueOf(val);
}
+ if (isGroupSearchEnabled() == false) {
+ groupSearchFirstEnabled = false;
+ }
return groupSearchFirstEnabled;
}
http://git-wip-us.apache.org/repos/asf/ranger/blob/6ec7d965/ugsync/src/test/java/org/apache/ranger/usergroupsync/TestLdapUserGroup.java
----------------------------------------------------------------------
diff --git a/ugsync/src/test/java/org/apache/ranger/usergroupsync/TestLdapUserGroup.java b/ugsync/src/test/java/org/apache/ranger/usergroupsync/TestLdapUserGroup.java
index b4c9065..e5bf110 100644
--- a/ugsync/src/test/java/org/apache/ranger/usergroupsync/TestLdapUserGroup.java
+++ b/ugsync/src/test/java/org/apache/ranger/usergroupsync/TestLdapUserGroup.java
@@ -248,7 +248,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
config.setGroupSearchEnabled(false);
- config.setGroupSearchFirstEnabled(false);
+ config.setGroupSearchFirstEnabled(true);
ldapBuilder.init();
PolicyMgrUserGroupBuilderTest sink = new PolicyMgrUserGroupBuilderTest();
sink.init();
@@ -304,6 +304,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
ldapBuilder.init();
PolicyMgrUserGroupBuilderTest sink = new PolicyMgrUserGroupBuilderTest();
@@ -322,6 +323,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
ldapBuilder.init();
@@ -341,6 +343,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
ldapBuilder.init();
@@ -359,6 +362,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setGroupSearchFilter("cn=*");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
ldapBuilder.init();
@@ -378,6 +382,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
ldapBuilder.init();
@@ -415,6 +420,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(false);
ldapBuilder.init();
@@ -434,6 +440,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(false);
ldapBuilder.init();
@@ -525,6 +532,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(false);
ldapBuilder.init();
@@ -605,6 +613,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
config.setDeltaSync(true);
@@ -626,6 +635,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(true);
config.setDeltaSync(true);
@@ -647,6 +657,7 @@ public class TestLdapUserGroup extends AbstractLdapTestUnit{
config.setUserGroupMemberAttributeName("member");
config.setUserObjectClass("organizationalPerson");
config.setGroupObjectClass("groupOfNames");
+ config.setGroupSearchEnabled(true);
config.setGroupSearchFirstEnabled(true);
config.setUserSearchEnabled(false);
config.setDeltaSync(true);