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 an...@apache.org on 2021/10/22 15:59:49 UTC
[jackrabbit-oak] branch trunk updated: OAK-9604 : Minor
improvements in o.a.j.oak.spi.security.authentication.external.impl.jmx
This is an automated email from the ASF dual-hosted git repository.
angela 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 ca2f8c2 OAK-9604 : Minor improvements in o.a.j.oak.spi.security.authentication.external.impl.jmx
ca2f8c2 is described below
commit ca2f8c2151ea546b95737af853f63afad613f954
Author: angela <an...@adobe.com>
AuthorDate: Fri Oct 22 17:59:01 2021 +0200
OAK-9604 : Minor improvements in o.a.j.oak.spi.security.authentication.external.impl.jmx
---
.../security/authentication/external/impl/jmx/ResultMessages.java | 6 +-----
.../security/authentication/external/impl/jmx/SyncMBeanImpl.java | 2 +-
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/ResultMessages.java b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/ResultMessages.java
index ec418b2..80308b4 100644
--- a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/ResultMessages.java
+++ b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/ResultMessages.java
@@ -22,16 +22,12 @@ import org.apache.jackrabbit.oak.spi.security.authentication.external.SyncResult
import org.apache.jackrabbit.oak.spi.security.authentication.external.SyncedIdentity;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
import java.util.ArrayList;
import java.util.List;
final class ResultMessages {
-
- private static final Logger log = LoggerFactory.getLogger(ResultMessages.class);
-
+
private final List<String> messages = new ArrayList<>();
ResultMessages() {}
diff --git a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SyncMBeanImpl.java b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SyncMBeanImpl.java
index 2841d85..a636947 100644
--- a/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SyncMBeanImpl.java
+++ b/oak-auth-external/src/main/java/org/apache/jackrabbit/oak/spi/security/authentication/external/impl/jmx/SyncMBeanImpl.java
@@ -65,7 +65,7 @@ public class SyncMBeanImpl implements SynchronizationMBean {
}
ExternalIdentityProvider idp = idpManager.getProvider(idpName);
if (idp == null) {
- log.error("No idp available for name", idpName);
+ log.error("No idp available for name {}", idpName);
throw new IllegalArgumentException("No idp manager available for name " + idpName);
}
return Delegatee.createInstance(repository, securityProvider, handler, idp);