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/19 21:04:52 UTC

incubator-ranger git commit: RANGER-278: reverting policy-validation-enable. after updating policy-validation to handle multiple mandatory resources at the same level, validation will be enabled

Repository: incubator-ranger
Updated Branches:
  refs/heads/master 0b7bc92bc -> 4a20a9dd9


RANGER-278: reverting policy-validation-enable. after updating policy-validation to handle multiple mandatory resources at the same level, validation will be enabled


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

Branch: refs/heads/master
Commit: 4a20a9dd92894d852a51f119be2966006fe1e752
Parents: 0b7bc92
Author: Madhan Neethiraj <ma...@apache.org>
Authored: Sun Apr 19 12:02:21 2015 -0700
Committer: Madhan Neethiraj <ma...@apache.org>
Committed: Sun Apr 19 12:02:21 2015 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/ranger/rest/ServiceREST.java   | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ranger/blob/4a20a9dd/security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java
----------------------------------------------------------------------
diff --git a/security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java b/security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java
index 01f2b7c..4ffbabd 100644
--- a/security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java
+++ b/security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java
@@ -864,8 +864,8 @@ public class ServiceREST {
 		RangerPolicy ret = null;
 		
 		try {
-			 RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
-			 validator.validate(policy, Action.CREATE, bizUtil.isAdmin());
+			 // RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
+			 // validator.validate(policy, Action.CREATE, bizUtil.isAdmin());
 
 			ensureAdminAccess(policy.getService(), policy.getResources());
 
@@ -898,8 +898,8 @@ public class ServiceREST {
 		RangerPolicy ret = null;
 
 		try {
-			 RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
-			 validator.validate(policy, Action.UPDATE, bizUtil.isAdmin());
+			 // RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
+			 // validator.validate(policy, Action.UPDATE, bizUtil.isAdmin());
 
 			ensureAdminAccess(policy.getService(), policy.getResources());
 
@@ -926,8 +926,8 @@ public class ServiceREST {
 		}
 
 		try {
-			 RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
-			 validator.validate(id, Action.DELETE);
+			 // RangerPolicyValidator validator = validatorFactory.getPolicyValidator(svcStore);
+			 // validator.validate(id, Action.DELETE);
 
 			RangerPolicy policy = svcStore.getPolicy(id);