[SLING-9449] Repoinit AclUtil#setPrincipalAcl throws exception if no path-based entry exists for principal

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: Repoinit JCR 1.1.26
• Fix Version/s: Repoinit JCR 1.1.30
• Component/s: Repoinit
• Assignee: Andrei Dulvac
• Reporter: Andrei Dulvac
• Created: 13/May/20 14:56
• Updated: 08/Apr/21 11:34
• Resolved: 13/May/20 15:11

Description

The check at https://github.com/apache/sling-org-apache-sling-jcr-repoinit/blob/master/src/main/java/org/apache/sling/jcr/repoinit/impl/AclUtil.java#L191 is very aggressive and results in an error getting thrown, which potentially causes the repository service to not start up.

It should be replaced with a log message.

Issue Links

• causes: SLING-10281 Add "ensure principal ACL" which throws an Exception it it fails
• relates to: SLING-9412 AclUtil.containsEquivalentEntry should not fail if no such node exists