Re: [MERGE] [3.1] Clean up commReset use in combination with tproxy or tcp_outgoing_address

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Tue, 11 May 2010 22:18:01 +1200

Henrik Nordström wrote:
> This patch backs out part of the patch for Bug #2222 and replaces it by
> crudely cycling over the available addresses, trying to skip over
> addresses not compatible with the current socket.
>
> This solves issues seen when using tproxy or tcp_outgoing_address and
> DNS of the requested host returns AAAA records in addition to A records.
>
> This change is interim, waiting for the larger connection setup
> overhaul. But seems to do the trick at least for me in tproxy & ipv4
> setups. Unfortunately I do not have an IPv6 connection at the moment to
> test IPv6 on, but I don't see how it could break IPv6.
>
> One effect of this change is that there will be no fallback to the other
> IP generation if the socket is configured to a specific outgoing
> address. Priorly the code threw away the outgoing address and tried
> again when encountering an incompatibility.
>

+1. IPv6 confirmed working.

The possible breakage was for v4/v6 gateways. My initial conversion
tests showed the socket was created from the client IP, inheriting the
TCP protocol family. That no longer seems to be true.

Amos

-- 
Please be using
   Current Stable Squid 2.7.STABLE9 or 3.1.3
Received on Tue May 11 2010 - 10:18:12 MDT

This archive was generated by hypermail 2.2.0 : Tue May 11 2010 - 12:00:08 MDT