Github user cshannon commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1816#discussion_r163924828
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/PostOfficeImpl.java
---
@@ -772,6 +772,8 @@ public RoutingStatus route(final Message message,
message.cleanupInternalProperties();
+ server.callBrokerPlugins(server.hasBrokerPlugins() ? plugin ->
plugin.beforeMessageRoute(message, context, direct, rejectDuplicates) : null);
--- End diff --
If there were messages sent to an address with no bindings the message
would get dropped because of lack of bindings. But with the old way the
message route methods were only triggered if the routing status was OK. With
the change the methods are called even if there are no bindings but the status
will say NO_BINDINGS, etc.
---
