You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2021/12/27 17:42:33 UTC

[logging-log4j2] 01/03: No need to nest.

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

ggregory pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git

commit 78eddfac376624c24d3b8dffb3f10e3ca8ff0824
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Mon Dec 27 10:33:15 2021 -0500

    No need to nest.
---
 .../java/org/apache/logging/log4j/core/appender/mom/JmsManager.java  | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/JmsManager.java b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/JmsManager.java
index beeb775..fec7fe0 100644
--- a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/JmsManager.java
+++ b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/JmsManager.java
@@ -131,10 +131,9 @@ public class JmsManager extends AbstractManager {
                     logger().error("Error creating JmsManager using JmsManagerConfiguration [{}]", data, e);
                     return null;
                 }
-            } else {
-                logger().error("JNDI must be enabled by setting log4j2.enableJndiJms=true");
-                return null;
             }
+            logger().error("JNDI must be enabled by setting log4j2.enableJndiJms=true");
+            return null;
         }
     }