Clarification in log message within ACL decorator

Jeff Bornemann · Jeff Bornemann · commit 2d314b694dc8 · 2017-01-06T12:25:09.000+07:00
diff --git a/src/main/groovy/com/twcable/grabbit/jcr/ACLProtoNodeDecorator.groovy b/src/main/groovy/com/twcable/grabbit/jcr/ACLProtoNodeDecorator.groovy
@@ -105,7 +105,7 @@ class ACLProtoNodeDecorator extends ProtoNodeDecorator {
         final String principalName = getPrincipalName(aceNode)
         Principal principal = getPrincipal(session, principalName)
         if(principal == null) {
-            log.warn "Principal for name ${principalName} does not exist, or is not accessible. Can not write ACE/ACL information."
+            log.warn "Principal for name ${principalName} does not exist, or is not accessible. Can not write ACE/ACL information for this principal on ${getParentPath()}. If this principal is currently being synched, it may not be accessible."
             return
         }
         Privilege[] privileges = getPrivilegeNames(aceNode).collect { String privilegeName ->

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ class ACLProtoNodeDecorator extends ProtoNodeDecorator {`
`105`	`105`	`final String principalName = getPrincipalName(aceNode)`
`106`	`106`	`Principal principal = getPrincipal(session, principalName)`
`107`	`107`	`if(principal == null) {`
`108`		`- log.warn "Principal for name ${principalName} does not exist, or is not accessible. Can not write ACE/ACL information."`
	`108`	`+ log.warn "Principal for name ${principalName} does not exist, or is not accessible. Can not write ACE/ACL information for this principal on ${getParentPath()}. If this principal is currently being synched, it may not be accessible."`
`109`	`109`	`return`
`110`	`110`	`}`
`111`	`111`	`Privilege[] privileges = getPrivilegeNames(aceNode).collect { String privilegeName ->`