[PATCH] Expose the request body length to vcl_recv{}

Poul-Henning Kamp phk at phk.freebsd.dk
Mon Apr 20 07:51:28 CEST 2015


--------
In message <CAJV_h0axRgUAa6TmngF_j=XOdPZs+SatfH7s6LnfYG++ZXNf4g at mail.gmail.com>
, Federico Schwindt writes:

>1. This will only set the length after std.cache_req_body(). We might want
>to return content-length if std.cache_req_body() was not previously called.
>I suppose some people might want to use it for other purposes than retrying
>and if someone forgets calling cache_req_body() he/she might mistakenly
>believe that there is no body. OTOH exposing this without curation might be
>a bad idea.

We need a magic value for "unknown" no matter what we do, with
chunked req.body we have no idea until it is cached.

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.



More information about the varnish-dev mailing list