<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 3, 2013 at 4:18 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">One last question then, why do you expect PATCH requests ?<br>
<br>
(I know I'm curious =)<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, May 3, 2013 at 10:12 PM, Sean Allen <<a href="mailto:sean@monkeysnatchbanana.com">sean@monkeysnatchbanana.com</a>> wrote:<br>
><br>
><br>
><br>
> On Fri, May 3, 2013 at 4:08 PM, Dridi Boukelmoune<br>
> <<a href="mailto:dridi.boukelmoune@zenika.com">dridi.boukelmoune@zenika.com</a>> wrote:<br>
>><br>
>> 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 <<br>
>> 400) {<br>
>>   ban("obj.http.x-url ~ " + beresp.http.x-invalidates);<br>
>> }<br>
>><br>
><br>
> Completeness I suppose. I see your point.<br>
> Something to think about.<br>
><br>
>><br>
>> 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>
><br>
><br>
> We don't have any clients that use PATCH. But that is a good point, I should<br>
> add just to be safe.<br></div></div></blockquote><div><br></div><div style>Ah, patch in the normal default. I put it in ours because we plan on switching over to using it</div><div style>instead of our existing hacky 'partial update PUT'</div>
</div><div><br></div>-- <br><div dir="ltr"><div><br></div>Ce n'est pas une signature<br></div>
</div></div>