Yep. The patch seems to be working correctly.
Thanks,
Sri
On Sat, Feb 12, 2011 at 1:21 PM, Amos Jeffries <squid3@xxxxxxxxxxxxx> wrote:
> On 12/02/11 22:05, Sri Rao wrote:
> <snip>
>>
>> Okay let's say I am trying to loadbalance using squid to 2 origin
>> servers. The 2 origin servers would be setup as cache_peers applying
>> the originserver directive no? Right now that wouldn't happen. It
>> would return not allowed for cache_peers right? The patch below would
>> allow for cache_peers if set as originserver to do the passthru you
>> are talking about above.
>>
>> I thought a possible patch could be:
>>
>> diff -Naur squid-3.1.11/src/tunnel.cc squid-3.1.11-cf/src/tunnel.cc
>> --- squid-3.1.11/src/tunnel.cc 2011-02-07 20:05:51.000000000 -0800
>> +++ squid-3.1.11-cf/src/tunnel.cc 2011-02-11 11:08:34.256181949
>> -0800
>> @@ -589,10 +589,10 @@
>> err->callback_data = tunnelState;
>> errorSend(tunnelState->client.fd(), err);
>> } else {
>> - if (tunnelState->servers->_peer)
>> - tunnelProxyConnected(tunnelState->server.fd(), tunnelState);
>> - else {
>> + if (!tunnelState->servers->_peer ||
>> tunnelState->servers->_peer->options.originserver)
>> tunnelConnected(tunnelState->server.fd(), tunnelState);
>> + else {
>> + tunnelProxyConnected(tunnelState->server.fd(), tunnelState);
>> }
>>
>> commSetTimeout(tunnelState->server.fd(),
>>
>> Wondering if there are reasons that this shouldn't be done?
>
> Hmm, my brain seems not to have been working much. :(
>
> Yes that appears a correct and useful solution. Thank you.
>
> If you can test this and verify that it produces the right operation for
> your needs I'll replace the earlier patch with this one.
>
> Amos
> --
> Please be using
> Current Stable Squid 2.7.STABLE9 or 3.1.11
> Beta testers wanted for 3.2.0.4
>
