Angela Schreiber created SLING-9959:
---------------------------------------
Summary: DefaultAclManager: path with create system user needs to
be intermediate path
Key: SLING-9959
URL: https://issues.apache.org/jira/browse/SLING-9959
Project: Sling
Issue Type: Bug
Components: Content-Package to Feature Model Converter
Reporter: Angela Schreiber
found the following repo-init statement to create system users:
https://github.com/apache/sling-org-apache-sling-feature-cpconverter/blob/master/src/main/java/org/apache/sling/feature/cpconverter/acl/DefaultAclManager.java#L109
which states:
{code}
formatter.format("create service user %s with path %s%n", systemUser.getId(),
systemUser.getPath());
{code}
now.... i might be mistaken, but if my reading of the {{SystemUserParser}} is
correct, the path defined with the {{SystemUser}} instance actually reflects
the path of the system user node itself:
https://github.com/apache/sling-org-apache-sling-feature-cpconverter/blob/master/src/main/java/org/apache/sling/feature/cpconverter/handlers/SystemUsersEntryHandler.java#L70-L75
if that was true the repo-init statements would be wrong, because the path
argument doesn't specify that path of the user node itself but rather the
intermediate path to be used.
so the statement should probably read:
{code}
formatter.format("create service user %s with path %s%n", systemUser.getId(),
systemUser.getParent().getPath());
{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
