Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1647#discussion_r149310333
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/AddressInfo.java
---
@@ -130,6 +130,10 @@ public String toString() {
}
public boolean isInternal() {
- return this.name.startsWith(ADVISORY_TOPIC);
+ return isInternal(this.name);
+ }
+
+ public static boolean isInternal(SimpleString address) {
--- End diff --
Shouldn't be needed should be an attribute on the AddressInfo
---
