You are viewing a plain text version of this content. The canonical link for it is here.
Posted to xmlrpc-auto@ws.apache.org by jo...@apache.org on 2007/10/27 20:30:02 UTC

svn commit: r589170 - in /webservices/xmlrpc/trunk: client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java src/changes/changes.xml

Author: jochen
Date: Sat Oct 27 11:30:01 2007
New Revision: 589170

URL: http://svn.apache.org/viewvc?rev=589170&view=rev
Log:
The XmlRpcSunHttpTransportFactory wasn't thread safe.
PR: XMLRPC-150

Modified:
    webservices/xmlrpc/trunk/client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java
    webservices/xmlrpc/trunk/src/changes/changes.xml

Modified: webservices/xmlrpc/trunk/client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java
URL: http://svn.apache.org/viewvc/webservices/xmlrpc/trunk/client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java?rev=589170&r1=589169&r2=589170&view=diff
==============================================================================
--- webservices/xmlrpc/trunk/client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java (original)
+++ webservices/xmlrpc/trunk/client/src/main/java/org/apache/xmlrpc/client/XmlRpcSunHttpTransportFactory.java Sat Oct 27 11:30:01 2007
@@ -22,15 +22,14 @@
  * {@link java.net.HttpURLConnection} class.
  */
 public class XmlRpcSunHttpTransportFactory extends XmlRpcTransportFactoryImpl {
-	private final XmlRpcSunHttpTransport HTTP_TRANSPORT;
-
 	/** Creates a new factory, which creates transports for the given client.
 	 * @param pClient The client, which is operating the factory.
 	 */
 	public XmlRpcSunHttpTransportFactory(XmlRpcClient pClient) {
 		super(pClient);
-		HTTP_TRANSPORT = new XmlRpcSunHttpTransport(pClient);
 	 }
 
-	public XmlRpcTransport getTransport() { return HTTP_TRANSPORT; }
+	public XmlRpcTransport getTransport() {
+	    return new XmlRpcSunHttpTransport(getClient());
+	}
 }

Modified: webservices/xmlrpc/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/webservices/xmlrpc/trunk/src/changes/changes.xml?rev=589170&r1=589169&r2=589170&view=diff
==============================================================================
--- webservices/xmlrpc/trunk/src/changes/changes.xml (original)
+++ webservices/xmlrpc/trunk/src/changes/changes.xml Sat Oct 27 11:30:01 2007
@@ -16,6 +16,9 @@
         Ensured that the server replies with a spec compliant request, if the client uses the
         content-length header.
       </action>
+      <action dev="jochen" type="fix" issue="XMLRPC-150">
+        The XmlRpcSunHttpTransportFactory wasn't thread safe.
+      </action>
     </release>
     <release version="3.1" date="2007-Aug-31">
       <action dev="jochen" type="add">