Uploaded image for project: 'ActiveMQ'
  1. ActiveMQ
  2. AMQ-6905

Resource Adapter clientId ActivationConfigProperty does conform to API document

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 5.15.3
    • 5.15.11, 5.16.0
    • RAR
    • None

    Description

      According to the following page http://activemq.apache.org/activation-spec-properties.html

      The clientId default is set by the resource adapter. This is not the case in the code.

      When defining the MDB to be Durable via subscriptionDurability, if the clientId is not configured an validation error is thrown stating "clientId must be set since durable subscription was requested".

      Looking at the code, if the documentation is correct, the ActiveMQActivationSpec validation method should no longer validate the clientId. The ActiveMQConnection makeConnection should change

      String clientId = activationSpec.getClientId();

      to

      String clientId = defaultValue(activationSpec.getClientId(), getInfo().getClientid());

       

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            jbonofre Jean-Baptiste Onofré
            dmerrittESP Darren Merritt
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 10m
                10m

                Slack

                  Issue deployment