[PATCH 3/3] Do not reset bereq on retry. Add rollback_bereq for this, purpose.

[Federico Schwindt]

Attached.

OK?




On Thu, Jul 24, 2014 at 12:08 PM, Poul-Henning Kamp <phk at phk.freebsd.dk>
wrote:

> In message <49589.1406199994 at critter.freebsd.dk>, "Poul-Henning Kamp"
> writes:
> >We chatted about this on IRC today, and this is just to keep everybody
> >in the loop:
> >
> >In general, I'm trying to slim down VCC to be "just the language" and
> >'rollback' is one of the remaining bits I'd like to get out of there,
> >so the new improved rollback goes into vmod.std.
> >
> >We leave the "rollback" which VCC implements alone for 4.x
> >
> >I have added a new VMOD data type "HTTP", which returns an entire
> >HTTP request/response (struct http *), so that somebody can now
> >implement the rollback functionality in vmod.std, so the end result
> >will become:
> >
> >       std.collect(req)
> >
> >       std.collect(bereq)
>
> and that of course should been:
>
>         std.rollback(req)
>         std.rollback(bereq)
>
>
> --
> Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
> phk at FreeBSD.ORG         | TCP/IP since RFC 956
> FreeBSD committer       | BSD since 4.3-tahoe
> Never attribute to malice what can adequately be explained by incompetence.
>
> _______________________________________________
> varnish-dev mailing list
> varnish-dev at varnish-cache.org
> https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20140724/336ed98f/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 000-std_rollback.patch
Type: text/x-diff
Size: 5441 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20140724/336ed98f/attachment.patch>