<div dir="ltr"><div><div><div>Since nitpicking is my name:<br><br>> +                       VSLb(bo->vsl, SLT_Uncacheable, "Illegal Vary header");<br><br></div>Before this there is a SLT_Error with "Illegal 'Vary' header from backend, making this a pass.". Repeating the reason here seems redundant.<br><br>> +       if (bo->do_pass) {<br>> +               bo->uncacheable = 1;                                            <br>> +               VSLb(bo->vsl, SLT_Uncacheable, "bereq");<br>> +       }<br><br></div>We could argue that this is uncacheable at the time we returned pass. Also "bereq" doesn't really tell us anything.<br></div>Furthermore, doing it in here will result in this being logged twice in some cases.<br><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 11, 2015 at 5:48 PM, Dridi Boukelmoune <span dir="ltr"><<a href="mailto:dridi@varni.sh" target="_blank">dridi@varni.sh</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> Either way, it might be ok, no further opinion really.<br>
<br>
</span>New patch based on your feedback :)<br>
</blockquote></div><br></div>