<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 3, 2013 at 4:08 PM, Dridi Boukelmoune <span dir="ltr"><<a href="mailto:dridi.boukelmoune@zenika.com" target="_blank">dridi.boukelmoune@zenika.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This is very interesting and didactic, and I have a few questions :<br>
<br>
Why not simply trust the backend?<br>
if (beresp.http.x-invalidates && beresp.status >= 200 && beresp.status < 400) {<br>
  ban("obj.http.x-url ~ " + beresp.http.x-invalidates);<br>
}<br>
<br></blockquote><div><br></div><div style>Completeness I suppose. I see your point.</div><div style>Something to think about.</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I don't see any handling of the PATCH method in vcl_recv, check it doesn't<br>
end up being piped:<br>
if (req.request != "GET" &&<br>
  req.request != "HEAD" &&<br>
  req.request != "PUT" &&<br>
  req.request != "POST" &&<br>
  req.request != "TRACE" &&<br>
  req.request != "OPTIONS" &&<br>
  req.request != "DELETE") {<br>
    /* Non-RFC2616 or CONNECT which is weird. */<br>
    return (pipe);<br>
}<br></blockquote><div><br></div><div style>We don't have any clients that use PATCH. But that is a good point, I should add just to be safe.</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Best Regards,<br>
Dridi<br>
<div><div class="h5"><br>
<br>
On Fri, May 3, 2013 at 9:15 PM, Sean Allen <<a href="mailto:sean@monkeysnatchbanana.com">sean@monkeysnatchbanana.com</a>> wrote:<br>
> Hopefully, some find it helpful.<br>
><br>
> I'm still fairly new to varnish. Improvements, suggestions etc welcomed.<br>
><br>
> <a href="http://dev.theladders.com/2013/05/varnish-in-five-acts/" target="_blank">http://dev.theladders.com/2013/05/varnish-in-five-acts/</a><br>
><br>
> --<br>
><br>
> Ce n'est pas une signature<br>
><br>
</div></div>> _______________________________________________<br>
> varnish-misc mailing list<br>
> <a href="mailto:varnish-misc@varnish-cache.org">varnish-misc@varnish-cache.org</a><br>
> <a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-misc" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-misc</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><br></div>Ce n'est pas une signature<br></div>
</div></div>