[ https://issues.apache.org/jira/browse/JCRVLT-111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15617748#comment-15617748 ]
Julian Sedding commented on JCRVLT-111: --------------------------------------- [~tripod] if you look at the implementation of TestVaultPackage it's exceedingly simple: it exposes a protected constructor accepting an {{Archive}} that is already present in ZipVaultPackage. Zipping the package on the fly probably requires more code and makes the tests slower. We could even consider providing a public constructor or utility to make testing in downstream projects easier. > Add support for o.a.j.api.security.authorization.PrincipalSetPolicy > ------------------------------------------------------------------- > > Key: JCRVLT-111 > URL: https://issues.apache.org/jira/browse/JCRVLT-111 > Project: Jackrabbit FileVault > Issue Type: New Feature > Reporter: angela > Assignee: Tobias Bocanegra > Fix For: 3.1.30 > > Attachments: JCRVLT-111.patch > > > jackrabbit API has been extended by an additional type of access control > policy, which isn't an ACL. fvault should be adjusted to be able to properly > import that type of access control policy. > as discussed: ac-handling {{MERGE}} and {{MERGE_PRESERVE}} should be > implemented the same way and just add extra principal names that are not yet > present in the set. -- This message was sent by Atlassian JIRA (v6.3.4#6332)