[Varnish] #1581: Duplicate SLT_Begin log records

[Varnish]

#1581: Duplicate SLT_Begin log records
----------------------+-----------------------------------------------
 Reporter:  daghf     |       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@…>):

 * owner:   => Martin Blix Grydeland <martin@…>
 * resolution:  worksforme => 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/1581#comment:3>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator