From ead13c11e557bb7a00e7e528a055067d7822f562 Mon Sep 17 00:00:00 2001 From: Anders Kaseorg Date: Sun, 11 Aug 2019 20:25:32 -0700 Subject: [PATCH] =?UTF-8?q?run-dev:=20Don=E2=80=99t=20rewrite=20the=20Cont?= =?UTF-8?q?ent-Length=20header.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit A HEAD response has a Content-Length but no body; it’s not correct in that case to let Tornado default Content-Length to 0. Signed-off-by: Anders Kaseorg --- tools/run-dev.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tools/run-dev.py b/tools/run-dev.py index 4b3e978bd3..c4c77beacf 100755 --- a/tools/run-dev.py +++ b/tools/run-dev.py @@ -302,12 +302,9 @@ class CombineHandler(BaseWebsocketHandler): self._headers = httputil.HTTPHeaders() # clear tornado default header for header, v in response.headers.get_all(): - if header != 'Content-Length': - # some header appear multiple times, eg 'Set-Cookie' - self.add_header(header, v) + # some header appear multiple times, eg 'Set-Cookie' + self.add_header(header, v) if response.body: - # rewrite Content-Length Header by the response - self.set_header('Content-Length', len(response.body)) self.write(response.body) self.finish()