You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@activemq.apache.org by "Lionel Cons (JIRA)" <ji...@apache.org> on 2018/06/21 12:57:00 UTC

[jira] [Created] (ARTEMIS-1950) Mismatch between durable-subscriber-name and durable-subscription-name

Lionel Cons created ARTEMIS-1950:
------------------------------------

             Summary: Mismatch between durable-subscriber-name and durable-subscription-name
                 Key: ARTEMIS-1950
                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1950
             Project: ActiveMQ Artemis
          Issue Type: Bug
            Reporter: Lionel Cons


The STOMP documentation describes the use of {{durable-subscription-name}} and does not mention {{durable-subscriber-name}} at all.

However, the source code ({{VersionedStompFrameHandler.java}}) contains:
{code}
      String durableSubscriptionName = frame.getHeader(Stomp.Headers.Subscribe.DURABLE_SUBSCRIBER_NAME);
      if (durableSubscriptionName == null) {
         durableSubscriptionName = frame.getHeader(Stomp.Headers.Subscribe.DURABLE_SUBSCRIPTION_NAME);
      }
      if (durableSubscriptionName == null) {
         durableSubscriptionName = frame.getHeader(Stomp.Headers.Subscribe.ACTIVEMQ_DURABLE_SUBSCRIPTION_NAME);
      }
{code}

In other words, if both {{durable-subscription-name}} and {{durable-subscriber-name}} are set, only the undocumented {{durable-subscriber-name}} is used.

Why do we have {{durable-subscriber-name}} in the first place?

If used, it should be documented. If not used, it should be removed.

BTW, {{activemq.subscriptionName}} is also used in the code and not documented. But at least it does not interfere with the documented {{durable-subscription-name}} as it comes after...



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)