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:03:29 UTC

[jackrabbit-oak] branch 1.22 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 1.22
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


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

commit 72ea92e42739e27f4eb645f737f0ecee939e70ec
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 2b2ae9eac3..ddac319585 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
@@ -551,7 +551,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) {