Re: [squid-users] Seemingly incorrect behavior: squid cache getting filled up on PUT requests

On 18/02/2014 1:23 p.m., Rajiv Desai wrote:
> I think I found the problem. This applies only to HTTPs traffic being
> cached with ssl-bump.
> Basically HttpRequest::maybeCacheable() does not check for PROTO_HTTPS
>
> Following patch fixes it:
>
> <patch>
> diff --git a/squid-3.HEAD-20140127-r13248/src/HttpRequest.cc
> b/squid-3.HEAD-20140127-r13248/src/HttpRequest.cc
> index dc18b33..ce6c411 100644
> --- a/squid-3.HEAD-20140127-r13248/src/HttpRequest.cc
> +++ b/squid-3.HEAD-20140127-r13248/src/HttpRequest.cc
> @@ -596,6 +596,7 @@ HttpRequest::maybeCacheable()
> switch (protocol) {
> case AnyP::PROTO_HTTP:
> + case AnyP::PROTO_HTTPS:
> if (!method.respMaybeCacheable())
> return false;
>
> </patch>
>
> -Rajiv

Aha! Thank you. Patch applied to Squid-3.

Amos
Received on Tue Feb 18 2014 - 02:12:39 MST