Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify underlying JCR session in documentation #220

Open
kwin opened this issue Sep 5, 2023 · 2 comments
Open

Clarify underlying JCR session in documentation #220

kwin opened this issue Sep 5, 2023 · 2 comments

Comments

@kwin
Copy link
Contributor

kwin commented Sep 5, 2023

For each of the execution possibilities the underlying JCR session/resource resolver should be clarified. I guess this is

  1. Startup hook: service resolver (which one, which permissions by default?)
  2. Install hook: service resolver (which one, which permissions by default?)
  3. Manual execution: requests based resolver (bound to the user session)
@kwin
Copy link
Contributor Author

kwin commented Sep 5, 2023

Particularly it is not clear when which of the three service users from

set ACL for aecu-content-migrator
allow jcr:all on /apps,/conf,/content,/etc,/home,/var
allow jcr:read on /
end
set ACL for aecu-admin
allow jcr:all on /
end
set ACL for aecu-service
allow jcr:read on /
allow jcr:all on /var/aecu
end
are used from where.

@nhirrle
Copy link
Collaborator

nhirrle commented Oct 17, 2023

Hi @kwin
First of all sorry for the late answer.
You are right, there are some improvements to do not only on the documentation side but also implementation side.
For the startup hook more permissions are required so the aecu-admin is used. For the manual execution within groovy indeed the user session should be used but it is not right now the case and instead a service user is used.
There is also protection on the Groovyconsole itself, usually a user with administration rights is able to execute groovyscripts only ( see https://github.com/orbinson/aem-groovy-console#osgi-configuration / Script Execution Allowed Groups).
Best, Nicolas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants