[PATCH] make bereq.uncacheable read-only

Federico Schwindt fgsch at lodoss.net
Wed Nov 5 14:23:20 CET 2014


Hi,

I've been spending time writing tests for some other work I'm doing and I
found bereq.uncacheable being writable, which doesn't make much sense to me.

The same can be achieved setting beresp.uncacheable in vcl_backend_response
or vcl_backend_error so this duplication seems unnecessary and having a
single place to set whether the object is uncacheable is better.

Comments? OK?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20141105/88f24cfe/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001.bereq_uncacheable.patch
Type: text/x-patch
Size: 1988 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20141105/88f24cfe/attachment.bin>


More information about the varnish-dev mailing list