You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ba...@apache.org on 2022/07/01 12:00:57 UTC

[jackrabbit-oak] branch trunk updated: OAK-9773: DefaultSyncContext#syncMembership() compares external ids case-sensitively

This is an automated email from the ASF dual-hosted git repository.

baedke pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 72fd3d043a OAK-9773: DefaultSyncContext#syncMembership() compares external ids case-sensitively
72fd3d043a is described below

commit 72fd3d043a76f3b5c8e72d79fdc0e9207f21bae7
Author: Manfred Baedke <ma...@greenbytes.de>
AuthorDate: Fri Jul 1 13:59:00 2022 +0200

    OAK-9773: DefaultSyncContext#syncMembership() compares external ids case-sensitively
    
    Fixed log output.
---
 .../spi/security/authentication/external/basic/DefaultSyncContext.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/basic/DefaultSyncContext.java b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/basic/DefaultSyncContext.java
index 0f61a1e0e1..a929e8e9a6 100644
--- a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/basic/DefaultSyncContext.java
+++ b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/basic/DefaultSyncContext.java
@@ -553,7 +553,7 @@ public class DefaultSyncContext implements SyncContext {
             boolean exists = grp != null;
 
             if (exists && !idMatches) {
-                log.warn("The existing authorizable {} and the external group {} have identifiers that only differ by case. Since the identifiers are compared case-insensitively, the existing authorizable will be considered to match the external group.");
+                log.warn("The existing group {} and the external group {} have identifiers that only differ by case. Since the identifiers are compared case-insensitively, the existing authorizable will be considered to match the external group.", grp.getID(), extGroup.getId());
             }
 
             if (!exists) {