[ 
https://issues.apache.org/jira/browse/SLING-10920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17444665#comment-17444665
 ] 

Angela Schreiber commented on SLING-10920:
------------------------------------------

[~karlpauls], the problem with service users coming from the seed or from other 
repo-init statements is that it's not possible to determine their path (in 
contrast to content packages). the path is used to determine if a given access 
control entry points to some user home.... but i agree that with the fix we 
essentially keep 2 sets of global service-user information: one 
'globalServiceUsers' for this defined in all packages and a second one for the 
ids of all those service users _plus_ the ids of those defined in repo-init and 
seed, for which we don't know the path. for the sake of clarity and 
maintainability i would suggest that we try to merge those 2 sets. i will 
create a separate improvement request for that.

> Converter doesn't reset the aclManager after first pass and rejects too much 
> after resets
> -----------------------------------------------------------------------------------------
>
>                 Key: SLING-10920
>                 URL: https://issues.apache.org/jira/browse/SLING-10920
>             Project: Sling
>          Issue Type: Bug
>          Components: Content-Package to Feature Model Converter
>    Affects Versions: Content-Package to Feature Model Converter 1.1.10
>            Reporter: Karl Pauls
>            Assignee: Karl Pauls
>            Priority: Major
>             Fix For: Content-Package to Feature Model Converter 1.1.12
>
>
> The converter doesn't reset the acl manager after the first pass. That makes 
> it so that the service users are collected and put into the all feature. 
> Furthermore, if it resets, it is loosing the acls that have matching service 
> users but that come too later in the contenpackage.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to