#13002 Fix "Force lookup if the request" block in varnish config or get rid of it
Closed: Fixed by kevin. Opened by adamwill.

The block starting:

    # Force lookup if the request is a no-cache request from the client.
#    if (req.http.Cache-Control ~ "no-cache") {
#        purge_url(req.url);

in ansible roles/varnish/templates/proxies.vcl.j2 was added in 0f43dd177d0b11085b4119c800326b7b8d65f75f (2014-12-08) by @pingou and commented out in f4d18014ce077071bf94df4e802183741b41fa34 (2015-01-20) by @kevin . It's been commented out ever since. Probably about time we either uncommented it or deleted it...


I've slept since then and don't recall why I disabled it. ;)

So, I guess I'd say lets drop it?

I guess the idea of most of the block was to try and avoid compressing already-compressed things, but no idea if that's still a going concern...

Not too sure what the purge_url bit was about.

Metadata Update from @phsmoura:
- Issue priority set to: Waiting on Assignee (was: Needs Review)
- Issue tagged with: medium-gain, medium-trouble, ops

I deleted it.

Metadata Update from @kevin:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

Metadata