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

Federico Schwindt fgsch at lodoss.net
Thu Jul 24 21:33:57 CEST 2014


fwiw, change ws0 to ws_bo in this diff.


On Thu, Jul 24, 2014 at 2:50 PM, Federico Schwindt <fgsch at lodoss.net> wrote:

> 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/cd773282/attachment.html>


More information about the varnish-dev mailing list