Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v4 0/3] Isolate If-Modified-Since handling in gitweb

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:53:24

"W. Trevor King" [off-list ref] writes:
On Tue, Mar 27, 2012 at 08:24:22PM +0100, Jakub Narebski wrote:
quoted
No, the status line was omitted unintentionally, thanks for fixing this.
I guess that web server (at least Apache) adds it if it is missing.
From the CGI 1.1 specs [1], section 9.2:

  If the script does not return a Status header, then "200 OK" should
  be assumed.

So, perhaps it is better to skip the Status header on success (less
for the server to parse).
I do not think it matters either way.  It is just a single header line, no?

The code is certainly more explicit with your patch.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help