Re: [PATCH] Compliance: rename Trailers header to Trailer everywhere.

From: Henrik Nordström <henrik_at_henriknordstrom.net>
Date: Tue, 24 Aug 2010 18:00:27 +0200

tis 2010-08-24 klockan 23:43 +1200 skrev Amos Jeffries:

> Given that Squid releases have been emitting the wrong header text for
> at least 4 years do you not think we should retain recognition of the
> incorrect name and upgrade to the correct one?

No. We have not been sending any Trailers headers. Only been stripping
the wrong header.

> This might be accomplished by having two texts pointing to HDR_TRAILER
> in src/HttpHeader.cc chunk @106:
>
> - {"Trailers", HDR_TRAILERS, ftStr},
> + {"Trailers", HDR_TRAILER, ftStr},
> + {"Trailer", HDR_TRAILER, ftStr},

No. Trailers != Trailer

Regards
Henrik
Received on Tue Aug 24 2010 - 16:00:32 MDT

This archive was generated by hypermail 2.2.0 : Wed Aug 25 2010 - 12:00:05 MDT