[ARTEMIS-1633] ActiveMQServerPlugin message routing callbacks should fire for all results

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.0
• Fix Version/s: 2.5.0
• Component/s: Broker
• Assignee: Christopher L. Shannon
• Reporter: Christopher L. Shannon
• Created: 24/Jan/18 16:02
• Updated: 08/Feb/18 12:10
• Resolved: 25/Jan/18 18:38

Description

When I added the callbacks for beforeMessageRoute and afterMessageRoute I accidentally inserted them so that they will only be called when the RoutingStatus result is OK.  They should always be called even if the status is no bindings, or no bindings DLA, etc to allow the plugin writer to handle events depending on the routing status.

Issue Links

• relates to: ARTEMIS-898 Artemis Plugin Support
• links to: GitHub Pull Request #1816