@@ -110,6 +110,10 @@ module JMX # # [:host] the host of the MBean server (defaults to "localhost") # [:port] the port of the MBean server (defaults to 3000) + # [:url] the url of the MBean server. + # No default. + # if the url is specified, the host & port parameters are + # not taken into account # [:username] the name of the user (if the MBean server requires authentication). # No default # [:password] the password of the user (if the MBean server requires authentication). @@ -125,7 +129,9 @@ module JMX password = args[:password] credentials = args[:credentials] - url = "service:jmx:rmi:///jndi/rmi://#{host}:#{port}/jmxrmi" + # host & port are not taken into account if url is set (see issue #7) + standard_url = "service:jmx:rmi:///jndi/rmi://#{host}:#{port}/jmxrmi" + url = args[:url] || standard_url unless credentials if !username.nil? and username.length > 0 lib/jmx4r.rb @@ -39,4 +39,27 @@ class TestConnection < Test::Unit::TestCase JConsole::stop 3001 end end + + def test_establish_connection_with_custom_url + begin + JConsole::start :port => 3001 + url = "service:jmx:rmi:///jndi/rmi://localhost:3001/jmxrmi" + JMX::MBean.establish_connection :url => url + ensure + JConsole::stop 3001 + end + end + + def test_establish_connection_with_custom_url_overrides_host_and_port + begin + JConsole::start :port => 3001 + good_url = "service:jmx:rmi:///jndi/rmi://localhost:3001/jmxrmi" + bad_port = 3000 + # specifying a :url discards the :port & :host parameters + JMX::MBean.establish_connection :port => bad_port, :url => good_url + ensure + JConsole::stop 3001 + end + end + end test/tc_connection.rb 3c77f7b265a08384e5e894bfe10a6b4d689001ca Jeff Mesnil jmesnil@gmail.com http://github.com/jmesnil/jmx4r/commit/07bc07affb65b83110ba27ebf13f3475e33855e4 07bc07affb65b83110ba27ebf13f3475e33855e4 2008-04-12T11:51:30-07:00 2008-04-12T11:51:30-07:00 fixed issue #7: Specify a URL when establishing a connection to the MBean server added :url arg to JMX::MBean.establish_connection + tests 23391673fddbbd2684bd480a5781caeaf6385c45 Jeff Mesnil jmesnil@gmail.com