[LOG4J2-367] JMS appenders send two messages for one append

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0-beta4, 2.0-beta5, 2.0-beta6, 2.0-beta7, 2.0-beta8
• Fix Version/s: 2.0-beta9
• Component/s: Appenders
• Labels: patch
• Assignee: Unassigned
• Reporter: David Parry
• Created: 23/Aug/13 12:34
• Updated: 23/Aug/13 13:11
• Resolved: 23/Aug/13 13:11

Description

When using the JMSTopicAppender or JMSQueueAppender calls to append result in JMSTopicManager calling send twice.

I can see that the code was refactored to support cleanup after exceptions, however the original call to send was not removed after a try/catch was introduced.

Index: core/src/main/java/org/apache/logging/log4j/core/net/JMSTopicManager.java
===================================================================
--- core/src/main/java/org/apache/logging/log4j/core/net/JMSTopicManager.java	(revision 1516828)
+++ core/src/main/java/org/apache/logging/log4j/core/net/JMSTopicManager.java	(revision )
@@ -104,7 +104,7 @@
         if (info == null) {
             info = connect(context, factoryBindingName, topicBindingName, userName, password, false);
         }
-        super.send(object, info.session, info.publisher);
+
         try {
             super.send(object, info.session, info.publisher);
         } catch (final Exception ex) {