Ticket #1272 (closed defect: fixed)

Opened 14 months ago

Last modified 8 months ago

duplicate Content-Length headers with pass and stream

Reported by: ehocdet Owned by: martin
Priority: normal Milestone:
Component: varnishd Version: 3.0.3
Severity: normal Keywords: stream Content-Length
Cc:

Description

In 3.0.3, i see a lot of duplicate Content-Length headers with this setup:

sub vcl_recv {

return (pass);

}

sub vcl_fetch {

set beresp.do_stream = true;

}

RES_StreamStart() add the second

Attachments

1272.patch Download (568 bytes) - added by ehocdet 13 months ago.
proposed fix
1272.2.patch Download (745 bytes) - added by ehocdet 12 months ago.

Change History

comment:1 Changed 14 months ago by martin

  • Owner set to martin

Changed 13 months ago by ehocdet

proposed fix

comment:2 Changed 12 months ago by ehocdet

A more generic fix in attachement (simply unset Content-Length).

Changed 12 months ago by ehocdet

comment:3 Changed 8 months ago by pada

This bug also occurs with version 3.0.4 and produces duplicate Content-Length headers, conflicting with  http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2

Patch  https://www.varnish-cache.org/trac/attachment/ticket/1272/1272.2.patch fixed it for us.

Please integrate into upstream.

comment:4 Changed 8 months ago by Martin Blix Grydeland <martin@…>

  • Status changed from new to closed
  • Resolution set to fixed

In [0a7e6caa0c6bd93003d4733e96f5ea054ac84cbc]:

Fix duplicate Content-Length headers with pass and stream

Fixes: #1272

Patch by: ehocdet

Note: See TracTickets for help on using tickets.