[ARTEMIS-1695] Improve Stomp backwards compatibility with 5.x

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.0
• Fix Version/s: 2.5.0
• Component/s: Broker, STOMP
• Assignee: Christopher L. Shannon
• Reporter: Christopher L. Shannon
• Created: 22/Feb/18 15:28
• Updated: 17/Mar/18 03:53
• Resolved: 23/Feb/18 14:18

Description

As I was testing out STOMP in Artemis vs ActiveMQ 5.x I found a few improvements to make:

 

1. ActiveMQ 5.x uses the header named "activemq.subscriptionName" for supplying a subscription name.  We should support that as well so clients don't need to change their code. 
2. Artemis doesn't create a session until the first message is published or a subscription is created.  I think we should create a session on connection after authentication is complete.  This is more in line with how CORE works and also when 5.x creates a session for a STOMP client.  
3. The Version 10 StompFrameHandler does not properly set the response to disconnect if there is an authentication error.

 

Also in ActiveMQ 5.x the prefixes "/topic/" and "/queue/" are used by default but this can already be configured by setting prefixes in Artemis so we don't need to worry about that.

Issue Links

• links to: GitHub Pull Request #1886