You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by ma...@apache.org on 2015/04/04 01:34:13 UTC

incubator-ranger git commit: RANGER-370: default policy for a new service should have isRecursive=true for resources that support recursive

Repository: incubator-ranger
Updated Branches:
  refs/heads/master b15105336 -> f5317ec95


RANGER-370: default policy for a new service should have isRecursive=true for resources that support recursive


Project: http://git-wip-us.apache.org/repos/asf/incubator-ranger/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-ranger/commit/f5317ec9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-ranger/tree/f5317ec9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-ranger/diff/f5317ec9

Branch: refs/heads/master
Commit: f5317ec954c49c74b207ef00b36e6028454810bb
Parents: b151053
Author: Madhan Neethiraj <ma...@apache.org>
Authored: Fri Apr 3 12:04:38 2015 -0700
Committer: Madhan Neethiraj <ma...@apache.org>
Committed: Fri Apr 3 16:33:51 2015 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/ranger/biz/ServiceDBStore.java  | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/f5317ec9/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java b/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
index 9fa9160..119ee64 100644
--- a/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
@@ -1018,13 +1018,16 @@ public class ServiceDBStore implements ServiceStore {
 			RangerPolicyResource polRes = new RangerPolicyResource();
 			polRes.setIsExcludes(false);
 			polRes.setIsRecursive(false);
-			
-			String value;
+
+			String value = "*";
 			if("path".equalsIgnoreCase(resDef.getName())) {
 				value = "/*";
-			} else {
-				value = "*";
 			}
+
+			if(resDef.getRecursivesupported()) {
+				polRes.setIsRecursive(Boolean.TRUE);
+			}
+
 			polRes.setValue(value);
 			resources.put(resDef.getName(), polRes);
 		}