<div dir="ltr">Hi,<div><br></div><div>Sounds like a good feature and shouldn't be too hard since we already have the VFPs<div>That said, if we were to implement it I reckon we should have both req.do_gunzip and req.do_gzip.</div></div><div><br></div><div>Best.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 10, 2017 at 4:33 PM,  <span dir="ltr"><<a href="mailto:jonathan.huot@thomsonreuters.com" target="_blank">jonathan.huot@thomsonreuters.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi varnish-dev,<br>
<br>
We're currently migrating our vmods from varnish 3, and we have a vmod which<br>
is uncompressing request's body to handle situation where backends are very old<br>
and not able to handle gzip encoding... It happens sometime.<br>
<br>
Instead of rewriting this vmod into a V4 module, I was guessing if you see the benefit of integrating into the core instead? It could be done by doing something like<br>
<br>
set req.do_gunzip = true;<br>
<br>
<br>
VTC could be approx..:<br>
<br>
$ cat g00008.vtc<br>
varnishtest "test basic gunzip for client request body"<br>
<br>
server s1 {<br>
        rxreq<br>
        expect req.http.content-encoding == <undef><br>
        txresp -status 200<br>
} -start<br>
<br>
varnish v1 -cliok "param.set http_gzip_support true" -vcl+backend {<br>
        sub vcl_recv {<br>
                set req.do_gunzip = true;<br>
        }<br>
} -start<br>
<br>
client c1 {<br>
        txreq -gzipbody {FOOBARFOO}<br>
        rxresp<br>
        expect resp.status == 200<br>
} -run<br>
<br>
varnish v1 -expect n_gzip == 0<br>
varnish v1 -expect n_gunzip == 1<br>
<br>
<br>
If you think it's interesting and any PR are welcomed, I will be glad to do one.<br>
Else, I will upgrade our VMOD instead :-)<br>
<br>
--<br>
Jonathan Huot<br>
Thomson Reuters<br>
<br>
<br>
______________________________<wbr>__<br>
<br>
This e-mail is for the sole use of the intended recipient and contains information that may be privileged and/or confidential. If you are not an intended recipient, please notify the sender by return e-mail and delete this e-mail and any attachments. Certain required legal entity disclosures can be accessed on our website.<<a href="http://site.thomsonreuters.com/site/disclosures/" rel="noreferrer" target="_blank">http://site.<wbr>thomsonreuters.com/site/<wbr>disclosures/</a>><br>
<br>
______________________________<wbr>_________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" rel="noreferrer" target="_blank">https://www.varnish-cache.org/<wbr>lists/mailman/listinfo/<wbr>varnish-dev</a><br>
</blockquote></div><br></div>