Apache
/
activemq-artemis
mirror of https://github.com/apache/activemq-artemis.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ARTEMIS-2289 SUBSCRIBE w/receipt returns 2 receipt-ids

This commit is contained in:
Justin Bertram 2019-03-30 14:36:02 -05:00 committed by Howard Gao
parent 8ca4b66e1d
commit 304ff79712
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
artemis-protocols/artemis-stomp-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/stomp/VersionedStompFrameHandler.java
View File

@ -126,9 +126,6 @@ public abstract class VersionedStompFrameHandler {
try {
StompPostReceiptFunction postProcessFunction = onSubscribe(request);
response = postprocess(request);
if (request.hasHeader(Stomp.Headers.RECEIPT_REQUESTED)) {
response.addHeader(Stomp.Headers.Response.RECEIPT_ID, request.getHeader(Stomp.Headers.RECEIPT_REQUESTED));
}
connection.sendFrame(response, postProcessFunction);
return null;
} catch (ActiveMQStompException e) {

11
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/stomp/v12/StompV12Test.java
View File

@ -33,6 +33,7 @@ import java.util.UUID;
import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicInteger;
import java.util.regex.Pattern;
import org.apache.activemq.artemis.api.core.RoutingType;
import org.apache.activemq.artemis.api.core.SimpleString;
@ -97,6 +98,16 @@ public class StompV12Test extends StompTestBase {
}
}
@Test
public void testSubscribeWithReceipt() throws Exception {
conn.connect(defUser, defPass);
Pattern p = Pattern.compile("receipt-id:.*\\nreceipt-id");
assertFalse(p.matcher(subscribe(conn, null).toString()).find());
conn.disconnect();
}
@Test
public void testConnection() throws Exception {
server.getSecurityStore().setSecurityEnabled(true);