[ 
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)

Reply via email to