[jira] [Created] (SLING-9959) DefaultAclManager: path with create system user needs to be intermediate path

[Angela Schreiber (Jira)]

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)