[AMQ-5719] Add clientId and subscriptionName to all durable-related exceptions and log messages within TopicRegion

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.11.1
• Fix Version/s: 5.12.0
• Component/s: Broker
• Labels: easyfix
• Assignee: Timothy A. Bish
• Reporter: Christopher L. Shannon
• Created: 13/Apr/15 20:19
• Updated: 14/Apr/15 21:43
• Resolved: 14/Apr/15 21:43

Description

In order to make it easier to debug durable related issues it would be useful if the error messages included the clientId and subscriptionName.

The specific use case for this was that a client tried to remove their durable subscription using the wrong clientId and received an error message which did not include the clientId so it was difficult to debug.

The error received just said: javax.jms.InvalidDestinationException: No durable subscription exists for: <subscriptionName>

It would be more useful if the error was: javax.jms.InvalidDestinationException: No durable subscription exists for clientId: <clientId> and subscriptionName: <subscriptionName>

I will be attaching a pull request to this ticket with the 2 specific exception messages I would like to see changed in TopicRegion.