You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hc.apache.org by mb...@apache.org on 2007/02/21 06:48:46 UTC

svn commit: r509901 - /jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java

Author: mbecke
Date: Tue Feb 20 21:48:46 2007
New Revision: 509901

URL: http://svn.apache.org/viewvc?view=rev&rev=509901
Log:
Added support for creating socks with the Scheme default port.

Modified:
    jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java

Modified: jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java?view=diff&rev=509901&r1=509900&r2=509901
==============================================================================
--- jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java (original)
+++ jakarta/httpcomponents/httpclient/trunk/src/java/org/apache/http/impl/conn/DefaultClientConnectionOperator.java Tue Feb 20 21:48:46 2007
@@ -135,7 +135,9 @@
         conn.announce(sock);
 
         sock = sf.connectSocket
-            (sock, target.getHostName(), target.getPort(), local, 0, params);
+            (sock, target.getHostName(), 
+                    target.getPort() == -1 ? schm.getDefaultPort() : target.getPort(), 
+                    local, 0, params);
         prepareSocket(sock, context, params);
 
         final boolean secure = sf.isSecure(sock);