Author: remy.maucherat(a)jboss.com Date: 2013-06-10 03:37:32 -0400 (Mon, 10 Jun 2013) New Revision: 2216 Modified: branches/8.0.x/src/main/java/org/apache/coyote/ajp/AjpAprProtocol.java branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11AprProtocol.java branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11NioProtocol.java Log: Port missing patch for APR and NIO connectors, for JMX names. Modified: branches/8.0.x/src/main/java/org/apache/coyote/ajp/AjpAprProtocol.java =================================================================== --- branches/8.0.x/src/main/java/org/apache/coyote/ajp/AjpAprProtocol.java 2013-06-05 11:28:21 UTC (rev 2215) +++ branches/8.0.x/src/main/java/org/apache/coyote/ajp/AjpAprProtocol.java 2013-06-10 07:37:32 UTC (rev 2216) @@ -166,14 +166,14 @@ if (this.domain != null ) { try { tpOname = new ObjectName - (domain + ":" + "type=ThreadPool,name=" + getName()); + (domain + ":" + "type=ThreadPool,name=" + getJmxName()); Registry.getRegistry(null, null) .registerComponent(endpoint, tpOname, null ); } catch (Exception e) { CoyoteLogger.AJP_LOGGER.errorRegisteringPool(e); } rgOname = new ObjectName - (domain + ":type=GlobalRequestProcessor,name=" + getName()); + (domain + ":type=GlobalRequestProcessor,name=" + getJmxName()); Registry.getRegistry(null, null).registerComponent(cHandler.global, rgOname, null); } } Modified: branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11AprProtocol.java =================================================================== --- branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11AprProtocol.java 2013-06-05 11:28:21 UTC (rev 2215) +++ branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11AprProtocol.java 2013-06-10 07:37:32 UTC (rev 2216) @@ -118,14 +118,14 @@ if( this.domain != null ) { try { tpOname=new ObjectName - (domain + ":" + "type=ThreadPool,name=" + getName()); + (domain + ":" + "type=ThreadPool,name=" + getJmxName()); Registry.getRegistry(null, null) .registerComponent(endpoint, tpOname, null ); } catch (Exception e) { CoyoteLogger.HTTP_LOGGER.errorRegisteringPool(e); } rgOname=new ObjectName - (domain + ":type=GlobalRequestProcessor,name=" + getName()); + (domain + ":type=GlobalRequestProcessor,name=" + getJmxName()); Registry.getRegistry(null, null).registerComponent ( cHandler.global, rgOname, null ); } Modified: branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11NioProtocol.java =================================================================== --- branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11NioProtocol.java 2013-06-05 11:28:21 UTC (rev 2215) +++ branches/8.0.x/src/main/java/org/apache/coyote/http11/Http11NioProtocol.java 2013-06-10 07:37:32 UTC (rev 2216) @@ -129,12 +129,12 @@ if (org.apache.tomcat.util.Constants.ENABLE_MODELER) { if (this.domain != null) { try { - tpOname = new ObjectName(domain + ":" + "type=ThreadPool,name=" + getName()); + tpOname = new ObjectName(domain + ":" + "type=ThreadPool,name=" + getJmxName()); Registry.getRegistry(null, null).registerComponent(endpoint, tpOname, null); } catch (Exception e) { CoyoteLogger.HTTP_LOGGER.errorRegisteringPool(e); } - rgOname = new ObjectName(domain + ":type=GlobalRequestProcessor,name=" + getName()); + rgOname = new ObjectName(domain + ":type=GlobalRequestProcessor,name=" + getJmxName()); Registry.getRegistry(null, null).registerComponent(cHandler.global, rgOname, null); } }