jackrabbit-oak-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ang...@apache.org
Subject svn commit: r1438069 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java
Date Thu, 24 Jan 2013 16:54:35 GMT
Author: angela
Date: Thu Jan 24 16:54:34 2013
New Revision: 1438069

URL: http://svn.apache.org/viewvc?rev=1438069&view=rev
Log:
OAK-51 : Access Control Management (WIP)

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java?rev=1438069&r1=1438068&r2=1438069&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java
(original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/authorization/AccessControlValidator.java
Thu Jan 24 16:54:34 2013
@@ -184,10 +184,8 @@ class AccessControlValidator implements 
         if (principalName == null || principalName.isEmpty()) {
             fail("Missing principal name.");
         }
-        // TODO
-        // if (!principalMgr.hasPrincipal(principal.getName())) {
-        //     throw new AccessControlException("Principal " + principal.getName() + " does
not exist.");
-        // }
+        // validity of principal is only a JCR specific contract and will not be
+        // enforced on the oak level.
     }
 
     private void checkValidPrivileges(String[] privilegeNames) throws CommitFailedException
{



Mime
View raw message