Review Request 31763: RANGER-278 Add validation for policy create/update/delete operations. Move action out of ctor to validate call, remove state from base valdiator.

Thu, 05 Mar 2015 10:12:13 -0800 

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31763/
-----------------------------------------------------------

Review request for ranger.


Bugs: RANGER-278
    https://issues.apache.org/jira/browse/RANGER-278


Repository: ranger


Description
-------

Added validation for policy create/update/delete operations. Moved action out 
of ctor to validate call.


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java 
ab8384c 
  
security-admin/src/main/java/org/apache/ranger/rest/RangerPolicyValidator.java 
PRE-CREATION 
  
security-admin/src/main/java/org/apache/ranger/rest/RangerServiceValidator.java 
08184c7 
  security-admin/src/main/java/org/apache/ranger/rest/RangerValidator.java 
3f25266 
  
security-admin/src/main/java/org/apache/ranger/rest/RangerValidatorFactory.java 
6c75a2f 
  security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java fc2178f 
  
security-admin/src/test/java/org/apache/ranger/rest/TestRangerPolicyValidator.java
 PRE-CREATION 
  
security-admin/src/test/java/org/apache/ranger/rest/TestRangerServiceValidator.java
 3bbb123 
  
security-admin/src/test/java/org/apache/ranger/rest/TestServiceRESTForValidation.java
 483e914 
  security-admin/src/test/java/org/apache/ranger/rest/TestServiceValidator.java 
a1879c4 
  security-admin/src/test/java/org/apache/ranger/rest/ValidationTestUtils.java 
0925aa1 

Diff: https://reviews.apache.org/r/31763/diff/


Testing
-------

Additional junits added.  Patch allies to latest trunk.


Thanks,

Alok Lal

Reply via email to