<div dir="ltr">They will need to be different subroutines though since the variables are different (resp vs beresp).<br><br>This means they need to be in sync which adds extra complexity and is error prone IMO.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 13, 2015 at 11:17 AM, 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">Hi,<br>
<br>
This is kind of like how you may duplicate bereq manipulations between<br>
vcl_miss and vcl_pass with Varnish 3 (no there's vcl_backend_fetch).<br>
<br>
A current workaround would be to call custom vcl subs in both<br>
vcl_synth and vcl_backend_error. Otherwise the patch looks simple<br>
enough :)<br>
<br>
Dridi<br>
<div><div class="h5"><br>
On Thu, Apr 9, 2015 at 10:55 PM, Federico Schwindt <<a href="mailto:fgsch@lodoss.net">fgsch@lodoss.net</a>> wrote:<br>
> Hi,<br>
><br>
> Currently if you want to have all your error handling in a single place,<br>
> i.e. vcl_synth{}, and you end up in vcl_backend_error{} you have to either<br>
> return retry and then abandon in vcl_backend_fetch{} or return deliver and<br>
> in vcl_deliver{} return synth.<br>
><br>
> I can't think of any reason for not allowing this so the attached patch<br>
> lifts this restriction.<br>
><br>
> Comments? OKs?<br>
><br>
</div></div>> _______________________________________________<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" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br>
</blockquote></div><br></div>