[master] 4ebc3cfec Make it possible to override the initial digest, and explain in a comment why it is not necessary.
Nils Goroll
nils.goroll at uplex.de
Tue Feb 16 03:45:37 UTC 2021
On 15/02/2021 18:50, Nils Goroll wrote:
> - Should we bump the vcl version?
>
> We have done so for comparably less significant changes (see introduction of
> UDS), and this definitely falls in the "semantic change without syntactic
> change" category.
>
> Existing VCL could, for example, rely on vcl_hash to be called.
Also this might contradict or at least complicate Dridis efforts to allow "other
version includes": For this vcl...
# included.vcl
vcl 4.1;
sub vcl_hash {
hash_data("I rely on it");
}
# default.vcl
vcl 4.2;
include "included.vcl";
sub vcl_recv {
hash_data(req.http.foo);
}
the whole point would be to _not_ silently allow the include: By the structure
of the include, the code in "included.vcl" would have precedence over anything
else, but with the hash_data change it now does not even get called any more.
It seems the hash change would have required more deliberation.
Nils
--
** * * UPLEX - Nils Goroll Systemoptimierung
Scheffelstraße 32
22301 Hamburg
tel +49 40 28805731
mob +49 170 2723133
fax +49 40 42949753
xmpp://slink@jabber.int.uplex.de/
http://uplex.de/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-commit/attachments/20210216/95da6086/attachment.bin>
More information about the varnish-commit
mailing list