[ https://issues.apache.org/jira/browse/SLING-10477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17360860#comment-17360860 ]
Angela Schreiber commented on SLING-10477: ------------------------------------------ [~kpauls] and myself were able to create a test that illustrates why the user-entry-handler in {{RecollectorVaultPackageScanner}} is required namely collection information about users/groups/system-users that allow to determine during the second phase if a given access control entry needs to be move to repo init (system user) or moved back to the package (users/groups). we additionally decided to add the group-entry-handler to make sure all principals defined in the content package(s) are identified during the first pass. > Review usage of UserEntryHandler in RecollectorVaultPackageScanner > ------------------------------------------------------------------ > > Key: SLING-10477 > URL: https://issues.apache.org/jira/browse/SLING-10477 > Project: Sling > Issue Type: Improvement > Components: Content-Package to Feature Model Converter > Reporter: Angela Schreiber > Priority: Major > > during testing of SLING-10468 we noticed that {{UserEntryHandler}} has been > added to the {{RecollectorVaultPackageScanner}} when SLING-8573 got fixed. it > required an extra check for assembler being null in > {{AbstractUserEntryHandler}}. > [~kpauls], lets review if that is really needed and add dedicated tests for > the original problem (in case they don't exist). -- This message was sent by Atlassian Jira (v8.3.4#803005)