[master] eb85bf87c vrt_filter: Minor polish

Nils Goroll nils.goroll at uplex.de
Thu Feb 13 14:49:56 UTC 2025


On 13.02.25 12:34, Dridi Boukelmoune wrote:
> 
> commit eb85bf87cae0b9feca74a20a1cee2b54a2b06d57
> Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
> Date:   Thu Feb 13 12:22:39 2025 +0100
> 
>      vrt_filter: Minor polish
>      
>      I'm not convinced by req_filter_can(). Filters have no effect on a
>      bereq, beresp or resp without a body, why should it be different for
>      req? We should be consistent here.

I would hope that your question was answered by the vcl_var.rst addition from 
00b190a0aed482a638fdb9f1207598cca3db6c27:

The req body (conversely to others) is one where VCL has control over when it 
gets processed by means of std.cache_req_body(): Filters act on the received 
body when that function is called, so confusion could be caused if people expect 
filters to have an effect when set/changed after the req.body is already cached.

To implement trailer support, I think we would need generic vcl "send/receive 
the body _now_" control, and this mechanism would get generalized.

> -	(void)bo;
> +	CHECK_OBJ_NOTNULL(bo, BUSYOBJ_MAGIC);

I am not against, but why null/magic check an unused value?

Otherwise thank you for the cstyle touchups.

Nils

-- 

Nils Goroll (he/him)

** * * 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: OpenPGP_0x1DCD8F57A3868BD7.asc
Type: application/pgp-keys
Size: 3943 bytes
Desc: OpenPGP public key
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-commit/attachments/20250213/43d19548/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-commit/attachments/20250213/43d19548/attachment-0003.bin>


More information about the varnish-commit mailing list