Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2376#discussion_r227610014
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/management/impl/view/ConsumerView.java
---
@@ -82,7 +82,7 @@ public Object getField(ServerConsumer consumer, String
fieldName) {
case "user":
return session.getUsername();
case "clientID":
- return consumer.getConnectionClientID();
+ return session.getMetaData("jms-client-id") == null ?
consumer.getConnectionClientID() : session.getMetaData("jms-client-id");
--- End diff --
this should not be needed, if anything the session and consumer models
should be re-worked to allow for a client id to be set at session level
generically.
---
