[ https://issues.apache.org/jira/browse/SLING-7960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16633980#comment-16633980 ]
Jörg Hoh edited comment on SLING-7960 at 10/1/18 1:20 PM: ---------------------------------------------------------- Yes, we are changing behavior of the implementation. To point out that in some cases the delete does not work ("delete user foo" does not delete a service, and "delete service user foo" does not delete a regular user") I added INFO statements (WARN is not justified IMO) in [^SLING-7960-3.patch] Regarding the "avoid surprises": If we would have a language specification for repo init, we could create a dependency towards it. If we don't want to do that, we could somehow add a dependency towards the jcr.repoinit module; for examples we could use OSGI capabilities to express the implementation level. was (Author: joerghoh): Yes, we are changing behavior of the implementation. To point out that in some cases the delete does not work ("delete user foo" does not delete a service, and "delete service user foo" does not delete a regular user") I added INFO statements (WARN is not justified IMO). Regarding the "avoid surprises": If we would have a language specification for repo init, we could create a dependency towards it. If we don't want to do that, we could somehow add a dependency towards the jcr.repoinit module; for examples we could use OSGI capabilities to express the implementation level. > Repoinit: "delete user" also deletes service user > ------------------------------------------------- > > Key: SLING-7960 > URL: https://issues.apache.org/jira/browse/SLING-7960 > Project: Sling > Issue Type: Bug > Components: Repoinit > Affects Versions: Repoinit JCR 1.1.8 > Reporter: Jörg Hoh > Priority: Major > Attachments: SLING-7960-2.patch, SLING-7960-3.patch, SLING-7960.patch > > > My current application uses a regular user, but with some changes to the > framework we need to migrate to regular service users. The service user is > being added as part of the deployment. > To automate this I decided to use the repoinit to remove the regular user and > then deploy the service user. Which works fine, but only until the next time > the repoinit is executed again. Because then the repoinit statement > {{delete user foo}} > purges the newly created serviceuser "foo". Which is totally unexpected, > because there is a dedicated command "delete service user" which is supposed > to remove users. So "delete user foo" must only delete a regular user "foo" > but not a service user named "foo". > -- This message was sent by Atlassian JIRA (v7.6.3#76005)