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 <[email protected]> Authored: Fri Apr 3 12:04:38 2015 -0700 Committer: Madhan Neethiraj <[email protected]> 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); }
