Search squid archive

Re: squid 5.0.4 cache_peer bug on https outgoing

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/28/20 5:39 AM, openwrt wrote:

> The bug is that cache_peer https CONNECT drops the port number

Please try the attached patch.

Thank you,

Alex.


> squid cache_peer https CONNECT packet.
> 
> CONNECT d.qqq.win  HTTP/1.1 (bad format: without port)
> 
> 0040   d1 d8 43 4f 4e 4e 45 43 54 20 64 2e 71 71 71 2e   ..CONNECT d.qqq.
> 0050   77 69 6e 20 48 54 54 50 2f 31 2e 31 0d 0a 55 73   win HTTP/1.1
Always send port in request-target of CONNECT requests to peers

Broken since inception (commit f5e1794 "Peering support for SslBump").

diff --git a/src/clients/HttpTunneler.cc b/src/clients/HttpTunneler.cc
index a4957f7..2743994 100644
--- a/src/clients/HttpTunneler.cc
+++ b/src/clients/HttpTunneler.cc
@@ -43,7 +43,7 @@ Http::Tunneler::Tunneler(const Comm::ConnectionPointer &conn, const HttpRequest:
     assert(connection);
     assert(callback);
     assert(dynamic_cast<Http::TunnelerAnswer *>(callback->getDialer()));
-    url = request->url.authority();
+    url = request->url.authority(true);
     watchForClosures();
 }
 
_______________________________________________
squid-users mailing list
squid-users@xxxxxxxxxxxxxxxxxxxxx
http://lists.squid-cache.org/listinfo/squid-users

[Index of Archives]     [Linux Audio Users]     [Samba]     [Big List of Linux Books]     [Linux USB]     [Yosemite News]

  Powered by Linux