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

rep:policy node in /apps not created by ui.apps package after split #2048

Closed
2 of 3 tasks
adamcin opened this issue Sep 19, 2019 · 0 comments
Closed
2 of 3 tasks

rep:policy node in /apps not created by ui.apps package after split #2048

adamcin opened this issue Sep 19, 2019 · 0 comments

Comments

@adamcin
Copy link
Contributor

adamcin commented Sep 19, 2019

Required Information

  • AEM Version, including Service Packs, Cumulative Fix Packs, etc: n/a
  • ACS AEM Commons Version: 4.3.2
  • Reproducible on Latest? yes

Expected Behavior

/apps/rep:policy should be created by ui.apps during oakpal scan to establish policies for acs-aem-commons system users.

Actual Behavior

from @ataimussova : There is an issue due to the split as rep:policy for apps are not being applied. The reason is that rep:policy is in ui.apps package and users are in ui.content. Due to the order of ui.apps being installed prior to ui.content, as a result system users have no access to /apps. It happens if you have no previous version of ACS AEM Commons installed because there are no ACS AEM Commons system users. Could you please fix?

Steps to Reproduce

I added a simple inline check to verify this situation with oakpal:

<check>
   <name>verify-acls</name>
   <inlineScript>
      function afterExtract(packageId, session) {
         if (!session.itemExists("/apps/rep:policy")) {
            oakpal.majorViolation("expected /apps/rep:policy node to be created", packageId);
         }
      }
   </inlineScript>
</check>

This is the result:

[INFO] --- oakpal-maven-plugin:1.4.2:verify (oakpal-verify) @ acs-aem-commons-ui.apps ---
[INFO] OakPAL Check Reports
[INFO]   verify-acls
[ERROR]    +- <MAJOR> expected /apps/rep:policy node to be created [acs-aem-commons-ui.apps-4.3.3-SNAPSHOT.zip]
[INFO]   verify-acls
[ERROR]    +- <MAJOR> expected /apps/rep:policy node to be created [acs-aem-commons-ui.apps-4.3.3-SNAPSHOT.zip]
[INFO]   verify-acls
[ERROR]    +- <MAJOR> expected /apps/rep:policy node to be created [acs-aem-commons-ui.apps-4.3.3-SNAPSHOT.zip]
[INFO]   verify-acls
[ERROR]    +- <MAJOR> expected /apps/rep:policy node to be created [acs-aem-commons-ui.apps-4.3.3-SNAPSHOT.zip]
[ERROR] ** Violations were reported at or above severity: MAJOR **
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

1 participant