This is an automated email from the ASF dual-hosted git repository.

brusdev pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/artemis-console.git


The following commit(s) were added to refs/heads/main by this push:
     new 6a9731d  ARTEMIS-5924 - Fix broker node display in JMX tree under 
heavy RBAC config
6a9731d is described below

commit 6a9731d5eb2ea67ae8922e58d41db0ee0fb3c38f
Author: Grzegorz Grzybek <[email protected]>
AuthorDate: Mon Mar 2 12:54:29 2026 +0100

    ARTEMIS-5924 - Fix broker node display in JMX tree under heavy RBAC config
---
 .../artemis-console-plugin/src/artemis-tree-processor.ts   | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git 
a/artemis-console-extension/artemis-extension/packages/artemis-console-plugin/src/artemis-tree-processor.ts
 
b/artemis-console-extension/artemis-extension/packages/artemis-console-plugin/src/artemis-tree-processor.ts
index 64862b3..5749ae2 100644
--- 
a/artemis-console-extension/artemis-extension/packages/artemis-console-plugin/src/artemis-tree-processor.ts
+++ 
b/artemis-console-extension/artemis-extension/packages/artemis-console-plugin/src/artemis-tree-processor.ts
@@ -90,9 +90,9 @@ function findBrokers(children: MBeanNode[] | undefined): 
MBeanNode[] {
   }
   const brokers: MBeanNode[] = []
   for (const c of children) {
-    if (c.id.endsWith('-folder')) {
-      if (oldBrokers.has(c.name)) {
-        const oldBrokerNode = oldBrokers.get(c.name)!
+    if (oldBrokers.has(c.name)) {
+      const oldBrokerNode = oldBrokers.get(c.name)!
+      if (c.id.endsWith('-folder')) {
         const newBrokerNode = new MBeanNode(null, `Broker ${c.name}`, true)
         newBrokerNode.icon = c.icon
         newBrokerNode.expandedIcon = c.expandedIcon
@@ -107,6 +107,14 @@ function findBrokers(children: MBeanNode[] | undefined): 
MBeanNode[] {
           newBrokerNode.adopt(c)
         })
         brokers.push(newBrokerNode)
+      } else {
+        // the broker doesn't have children at all, but we still need the node
+        const newBrokerNode = new MBeanNode(null, `Broker ${c.name}`, false)
+        newBrokerNode.icon = c.icon
+        newBrokerNode.objectName = oldBrokerNode.objectName
+        newBrokerNode.mbean = oldBrokerNode.mbean
+        newBrokerNode.propertyList = oldBrokerNode.propertyList
+        brokers.push(newBrokerNode)
       }
     }
   }


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to