You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dw...@apache.org on 2009/01/22 19:13:19 UTC
svn commit: r736731 - in /geronimo/server/branches/2.1/framework/modules:
geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/
geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/
Author: dwoods
Date: Thu Jan 22 10:13:19 2009
New Revision: 736731
URL: http://svn.apache.org/viewvc?rev=736731&view=rev
Log:
GERONIMO-4518 revert changes mde in Rev736044
Modified:
geronimo/server/branches/2.1/framework/modules/geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/JMXConnector.java
geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/GeronimoRMIClientSocketFactory.java
Modified: geronimo/server/branches/2.1/framework/modules/geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/JMXConnector.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/framework/modules/geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/JMXConnector.java?rev=736731&r1=736730&r2=736731&view=diff
==============================================================================
--- geronimo/server/branches/2.1/framework/modules/geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/JMXConnector.java (original)
+++ geronimo/server/branches/2.1/framework/modules/geronimo-jmx-remoting/src/main/java/org/apache/geronimo/jmxremoting/JMXConnector.java Thu Jan 22 10:13:19 2009
@@ -188,7 +188,7 @@
} else {
log.warn("Starting unauthenticating JMXConnector for " + jmxServiceURL);
}
- RMIClientSocketFactory socketFactory = new GeronimoRMIClientSocketFactory(host, 2 * 60 * 1000, 5 * 60 * 1000);
+ RMIClientSocketFactory socketFactory = new GeronimoRMIClientSocketFactory(2 * 60 * 1000, 5 * 60 * 1000);
env.put(RMIConnectorServer.RMI_CLIENT_SOCKET_FACTORY_ATTRIBUTE, socketFactory);
RMIServerSocketFactory serverSocketFactory = new GeronimoRMIServerSocketFactory(host);
env.put(RMIConnectorServer.RMI_SERVER_SOCKET_FACTORY_ATTRIBUTE, serverSocketFactory);
Modified: geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/GeronimoRMIClientSocketFactory.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/GeronimoRMIClientSocketFactory.java?rev=736731&r1=736730&r2=736731&view=diff
==============================================================================
--- geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/GeronimoRMIClientSocketFactory.java (original)
+++ geronimo/server/branches/2.1/framework/modules/geronimo-kernel/src/main/java/org/apache/geronimo/kernel/rmi/GeronimoRMIClientSocketFactory.java Thu Jan 22 10:13:19 2009
@@ -31,20 +31,15 @@
private int connectionTimeout = -1;
private int readTimeout = -1;
- private String host=null;
- public GeronimoRMIClientSocketFactory(String _host,int connectionTimeout, int readTimeout) {
- this.host=_host;
+ public GeronimoRMIClientSocketFactory(int connectionTimeout, int readTimeout) {
this.connectionTimeout = connectionTimeout;
this.readTimeout = readTimeout;
}
- public Socket createSocket(String _host, int port) throws IOException {
+ public Socket createSocket(String host, int port) throws IOException {
Socket socket = new Socket();
socket.bind(null);
- if(host==null){
- host=_host;
- }
socket.connect(new InetSocketAddress(host, port), (this.connectionTimeout > 0) ? this.connectionTimeout : 0);
if (this.readTimeout >= 0) {
socket.setSoTimeout(this.readTimeout);