jetty-9 improved mbean names

2 files changed, 32 insertions, 21 deletions

diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/handler/jmx/AbstractHandlerMBean.java b/jetty-server/src/main/java/org/eclipse/jetty/server/handler/jmx/AbstractHandlerMBean.java
index c0380eb680..b4976299da 100644
--- a/jetty-server/src/main/java/org/eclipse/jetty/server/handler/jmx/AbstractHandlerMBean.java
+++ b/jetty-server/src/main/java/org/eclipse/jetty/server/handler/jmx/AbstractHandlerMBean.java
@@ -47,7 +47,12 @@ public class AbstractHandlerMBean extends ObjectMBean
             String basis = null;
             if (_managed instanceof ContextHandler)
             {
-                return null;
+                ContextHandler handler = (ContextHandler)_managed;
+                String context = getContextName(handler);
+                if (context==null)
+                    context=handler.getDisplayName();
+                if (context!=null)
+                    return context;
             }
             else if (_managed instanceof AbstractHandler)
             {
@@ -68,26 +73,6 @@ public class AbstractHandlerMBean extends ObjectMBean
         }
         return super.getObjectContextBasis();
     }
-    
-    /* ------------------------------------------------------------ */
-    @Override
-    public String getObjectNameBasis()
-    {
-        if (_managed != null )
-        {
-            String name = null;
-            if (_managed instanceof ContextHandler)
-            {
-                ContextHandler context = (ContextHandler)_managed;
-                name = getContextName(context);
-            }
-            
-            if (name != null)
-                return name;
-        }
-        
-        return super.getObjectNameBasis();
-    }
 
     /* ------------------------------------------------------------ */
     protected String getContextName(ContextHandler context)
@@ -116,6 +101,9 @@ public class AbstractHandlerMBean extends ObjectMBean
             }
         }
         
+        if (context.getVirtualHosts()!=null && context.getVirtualHosts().length>0)
+            name='"'+name+"@"+context.getVirtualHosts()[0]+'"';
+        
         return name;
     }
 }
diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/jmx/AbstractConnectorMBean.java b/jetty-server/src/main/java/org/eclipse/jetty/server/jmx/AbstractConnectorMBean.java
new file mode 100644
index 0000000000..af28bb0826
--- /dev/null
+++ b/jetty-server/src/main/java/org/eclipse/jetty/server/jmx/AbstractConnectorMBean.java
@@ -0,0 +1,23 @@
+package org.eclipse.jetty.server.jmx;
+
+import org.eclipse.jetty.jmx.ObjectMBean;
+import org.eclipse.jetty.server.AbstractConnector;
+import org.eclipse.jetty.util.annotation.ManagedObject;
+
+@ManagedObject("MBean Wrapper for Connectors")
+public class AbstractConnectorMBean extends ObjectMBean
+{
+    final AbstractConnector _connector;
+    public AbstractConnectorMBean(Object managedObject)
+    {
+        super(managedObject);
+        _connector=(AbstractConnector)managedObject;
+    }
+    @Override
+    public String getObjectContextBasis()
+    {
+        return String.format("%s@%x",_connector.getDefaultProtocol(),_connector.hashCode());
+    }
+    
+    
+}