On 11/30/2016 04:44 AM, Garri Djavadyan wrote: > * If-Modified-Since header is ignored if If-None-Match header exists > (RFC7232 compliance)
> if (r.header.has(Http::HdrType::IF_NONE_MATCH)) { > + if (e->hasIfNoneMatchEtag(r)) { > + // RFC 7232: If-None-Match recipient MUST ignore IMS > r.flags.ims = false; > r.ims = -1; > r.imslen = 0; > r.header.delById(Http::HdrType::IF_MODIFIED_SINCE); > > sendNotModifiedOrPreconditionFailedError(); > return true; > } > > + // If-None-Match did not match; treat as an unconditional hit > + return false; > } Can you explain why we are not clearing various IMS flags and headers when hasIfNoneMatchEtag() returns false? Does not RFC7232 compliance require equal treatment of IMS in both true and false cases? Thank you, Alex. _______________________________________________ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev