[Varnish] #1588: Timestamp records are recorded before SLT_Begin on pipelining requests
Varnish
varnish-bugs at varnish-cache.org
Tue Sep 9 14:29:16 CEST 2014
#1588: Timestamp records are recorded before SLT_Begin on pipelining requests
----------------------+-----------------------------------------------
Reporter: martin | Owner: Martin Blix Grydeland <martin@…>
Type: defect | Status: closed
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: normal | Resolution: fixed
Keywords: |
----------------------+-----------------------------------------------
Changes (by Martin Blix Grydeland <martin@…>):
* status: new => closed
* owner: => Martin Blix Grydeland <martin@…>
* resolution: => fixed
Comment:
In [51e13a4a5e3940c29730ed65aa0b7b482ba21e61]:
{{{
#!CommitTicketReference repository=""
revision="51e13a4a5e3940c29730ed65aa0b7b482ba21e61"
Allocate request VXID only when it's needed and delay all logging
until it's been allocated.
This patch delays the VXID allocation until the time we know that we
will actually be logging anything for this request (not completely all
whitespace noise). This keeps the code in one place only (plus the ESI
mock request handling), instead of separated between the linger wait,
cache acceptor and pipelining code paths.
Coincidentally fixes the logging of timestamps problem in #1588.
Test case by daghf.
Fixes: #1581 #1588
}}}
--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1588#comment:1>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator
More information about the varnish-bugs
mailing list