createConnector="false" has no effect on Tiger

• Key: AMQ-950
• Type: Bug
• Status: Resolved
• Resolution: Fixed
• Priority: Minor
• Assignee: Unassigned
• Reporter: Renaud Bruyeron
• Component/s: Broker
• Affects Version/s: 4.1.0
• Fix Version/s: 4.1.0

Description

On Tiger, activemq always creates a rmi connector on port 1099 no matter what I do with -Djavax.management... and <managementContext/>
In particular, setting createConnector="false" should prevent AMQ from setting up its own connector, but it does not.

The problem is in the findMBeanServer() method:
if (result == null && createMBeanServer) { result = createMBeanServer(); }
else { createConnector(result); }

result is not null on Tiger with useJmx="true", and createConnector is not protected by if(createConnector) like it is on the non-Tiger flow.
The fix (I think) is simply to do this:
if (result == null && createMBeanServer) { result = createMBeanServer(); } }
else {
if(createConnector){ createConnector(result); }
}