[activemq-artemis] branch master updated: ARTEMIS-2650 OpenWire Browsing is not sending credits

[cl...@apache.org]

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

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


The following commit(s) were added to refs/heads/master by this push:
     new d15b3ae  ARTEMIS-2650 OpenWire Browsing is not sending credits
     new abbadb3  This closes #3020
d15b3ae is described below

commit d15b3aeed6131bc6e199a470a26104a9ddfb18e7
Author: Clebert Suconic <cl...@apache.org>
AuthorDate: Mon Mar 16 09:09:33 2020 -0400

    ARTEMIS-2650 OpenWire Browsing is not sending credits
---
 .../apache/activemq/artemis/core/protocol/openwire/amq/AMQConsumer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/amq/AMQConsumer.java b/artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/amq/AMQConsumer.java
index b5e6fdc..57d0067 100644
--- a/artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/amq/AMQConsumer.java
+++ b/artemis-protocols/artemis-openwire-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/openwire/amq/AMQConsumer.java
@@ -304,7 +304,7 @@ public class AMQConsumer {
 
       List<MessageReference> ackList = serverConsumer.getDeliveringReferencesBasedOnProtocol(removeReferences, first, last);
 
-      if (ack.isIndividualAck() || ack.isStandardAck() || ack.isPoisonAck()) {
+      if (removeReferences && (ack.isIndividualAck() || ack.isStandardAck() || ack.isPoisonAck())) {
          acquireCredit(ackList.size());
       } else {
          acquireCredit(ack.getMessageCount());