[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