<div dir="ltr"><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>On Wed, Jul 6, 2022 at 12:07 AM Dridi Boukelmoune <<a href="mailto:dridi@varni.sh">dridi@varni.sh</a>> wrote:</div></div></div></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I didn't find how to scam people with NFTs in the manual, should I<br>
open a github issue?<br></blockquote><div><br></div><div>No no no, it's just to weed out the weak investors, send me 5 bitcoins and I'll show where it is.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
In general, I agree, the API looks rather well thought out, even<br>
though it does suffer bloated constructor syndrome.</blockquote><div><br></div><div>Yes, I realized later that I could use the event method of the backend to finalize the object, however, I'm not sure this:</div><div><br></div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div class="gmail_quote"><div><font face="monospace">new client = reqwest.client();</font></div></div><div class="gmail_quote"><div><font face="monospace">client.set_base_url("<a href="http://www.example.com">http://www.example.com</a>");</font></div></div><div class="gmail_quote"><div><font face="monospace">client.set_follow(5);</font></div></div><div class="gmail_quote"><div><font face="monospace">client.set_brotli(true);</font></div></div><div class="gmail_quote"><div><font face="monospace">client.set_probe(p1);</font></div></div><div class="gmail_quote"><div><font face="monospace">client.set_<span style="color:rgb(36,41,47);font-size:12px;white-space:pre">connect_timeout(5s);</span></font></div></div></blockquote><div class="gmail_quote"><div><span style="color:rgb(36,41,47);font-family:ui-monospace,SFMono-Regular,"SF Mono",Menlo,Consolas,"Liberation Mono",monospace;font-size:12px;white-space:pre"><br></span></div><div><font color="#24292f" face="arial, sans-serif"><span style="font-size:12px;white-space:pre">is more readable, or practical than:</span></font></div><div><font color="#24292f" face="arial, sans-serif"><span style="font-size:12px;white-space:pre"><br></span></font></div></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div class="gmail_quote"><div><font color="#24292f" face="monospace"><span style="font-size:12px;white-space:pre">new client = reqwest.client(</span></font></div></div><div class="gmail_quote"><div><font face="monospace"><font color="#24292f"><span style="font-size:12px;white-space:pre">    base_url = "</span></font><a href="http://www.example.com">http://www.example.com</a><span style="font-size:12px;white-space:pre;color:rgb(36,41,47)">",</span></font></div></div><div class="gmail_quote"><div><span style="font-size:12px;white-space:pre;color:rgb(36,41,47)"><font face="monospace">    follow = 5,</font></span></div></div><div class="gmail_quote"><div><span style="font-size:12px;white-space:pre;color:rgb(36,41,47)"><font face="monospace">    auto_brotli = true,</font></span></div></div><div class="gmail_quote"><div><span style="font-size:12px;white-space:pre;color:rgb(36,41,47)"><font face="monospace">    probe = p1,</font></span></div></div><div class="gmail_quote"><div><span style="font-size:12px;white-space:pre;color:rgb(36,41,47)"><font face="monospace">    connect_timeout = 5s</font></span></div></div><div class="gmail_quote"><div><font color="#24292f" face="monospace"><span style="font-size:12px;white-space:pre">);</span></font></div></div></blockquote><div class="gmail_quote"><div> </div><div>(consider this a question to you all, if you have an opinion, voice it!)</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Did you put only<br>
timeout and connect_timeout to lower the number of arguments or<br>
weren't you able to implement ftbo and bbto with reqwest? I suspect<br>
both :p<br></blockquote><div><br></div><div>Definitely the latter, once you pass the 6-7 arguments threshod, the sky's the limit.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Also it says this:<br>
<br>
> In practice, when contacting a backend, you will need to `unset bereq.http.accept-encoding;`, as Varnish sets it automatically.<br>
<br>
Probably a nice spot to mention<br>
<a href="https://varnish-cache.org/docs/7.0/reference/varnishd.html#http-gzip-support" rel="noreferrer" target="_blank">https://varnish-cache.org/docs/7.0/reference/varnishd.html#http-gzip-support</a><br>
to explain why one would be set.<br>
<br>
On the other hand, if you disable gzip support you may also be<br>
forwarding the client's accept-encoding header if it survived all the<br>
way to the backend fetch<br></blockquote><div><br></div><div>Good points, I can update the docs. I'm wondering though if it's better to special-case the AE header handling in the vmod and try to be smart, or just let the user do it in VCL...</div><div> </div><div>-- </div><div>Guillaume Quintard<br></div></div></div>