Skip to content
Permalink
Browse files
OAK-9779 : PermissionConstants.PERMISSION_PROPERTY_NAMES does not lis…
…t rep:isAllow
  • Loading branch information
anchela committed May 24, 2022
1 parent c5eb75a commit 4371869bb6f67e054cc53ddc592e7e528bc6cb0f
Showing 2 changed files with 3 additions and 2 deletions.
@@ -43,7 +43,7 @@ public interface PermissionConstants {

Set<String> PERMISSION_NODETYPE_NAMES = ImmutableSet.of(NT_REP_PERMISSIONS, NT_REP_PERMISSION_STORE);
Set<String> PERMISSION_NODE_NAMES = ImmutableSet.of(REP_PERMISSION_STORE);
Set<String> PERMISSION_PROPERTY_NAMES = ImmutableSet.of(REP_ACCESS_CONTROLLED_PATH, REP_PRIVILEGE_BITS);
Set<String> PERMISSION_PROPERTY_NAMES = ImmutableSet.of(REP_ACCESS_CONTROLLED_PATH, REP_PRIVILEGE_BITS, REP_IS_ALLOW);

/**
* Configuration parameter to enforce backwards compatible permission
@@ -25,6 +25,7 @@
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.NT_REP_PERMISSIONS;
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.NT_REP_PERMISSION_STORE;
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.REP_ACCESS_CONTROLLED_PATH;
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.REP_IS_ALLOW;
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.REP_PERMISSION_STORE;
import static org.apache.jackrabbit.oak.spi.security.authorization.permission.PermissionConstants.REP_PRIVILEGE_BITS;
import static org.junit.Assert.assertEquals;
@@ -43,7 +44,7 @@ public void testNodeNames() {

@Test
public void testPropertyNames() {
assertEquals(ImmutableSet.of(REP_ACCESS_CONTROLLED_PATH, REP_PRIVILEGE_BITS), PermissionConstants.PERMISSION_PROPERTY_NAMES);
assertEquals(ImmutableSet.of(REP_ACCESS_CONTROLLED_PATH, REP_PRIVILEGE_BITS, REP_IS_ALLOW), PermissionConstants.PERMISSION_PROPERTY_NAMES);
}

@Test

0 comments on commit 4371869

Please sign in to comment.