<div dir="ltr">Right, Arianna pointed that too. She suggested:<br><br>if (ctx->req->req_body_status != REQ_BODY_CACHED) {<br>
   VSLb(ctx->vsl, SLT_VCL_Error, "Unbuffered req.body");<br>
   return (-1);<br>
 }<br>
<br>Do we want to do something like that only if content_length is 0 or regardless if it's not buffered?<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 6:51 AM, Poul-Henning Kamp <span dir="ltr"><<a href="mailto:phk@phk.freebsd.dk" target="_blank">phk@phk.freebsd.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">--------<br>
In message <CAJV_h0axRgUAa6TmngF_j=<a href="mailto:XOdPZs%2BSatfH7s6LnfYG%2B%2BZXNf4g@mail.gmail.com">XOdPZs+SatfH7s6LnfYG++ZXNf4g@mail.gmail.com</a>><br>
<span class="">, Federico Schwindt writes:<br>
<br>
>1. This will only set the length after std.cache_req_body(). We might want<br>
>to return content-length if std.cache_req_body() was not previously called.<br>
>I suppose some people might want to use it for other purposes than retrying<br>
>and if someone forgets calling cache_req_body() he/she might mistakenly<br>
>believe that there is no body. OTOH exposing this without curation might be<br>
>a bad idea.<br>
<br>
</span>We need a magic value for "unknown" no matter what we do, with<br>
chunked req.body we have no idea until it is cached.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20<br>
phk@FreeBSD.ORG         | TCP/IP since RFC 956<br>
FreeBSD committer       | BSD since 4.3-tahoe<br>
Never attribute to malice what can adequately be explained by incompetence.<br>
</font></span></blockquote></div><br></div>