[nginx] svn commit: r4374 - trunk/src/http/modules

mdounin at mdounin.ru mdounin at mdounin.ru
Mon Dec 19 11:25:41 UTC 2011


Author: mdounin
Date: 2011-12-19 11:25:40 +0000 (Mon, 19 Dec 2011)
New Revision: 4374

Log:
Scgi: removed error if there is no Status header.

The SCGI specification doesn't specify format of the response, and assuming
CGI specs should be used there is no reason to complain.  RFC 3875
explicitly states that "A Status header field is optional, and status
200 'OK' is assumed if it is omitted".


Modified:
   trunk/src/http/modules/ngx_http_scgi_module.c

Modified: trunk/src/http/modules/ngx_http_scgi_module.c
===================================================================
--- trunk/src/http/modules/ngx_http_scgi_module.c	2011-12-19 11:24:32 UTC (rev 4373)
+++ trunk/src/http/modules/ngx_http_scgi_module.c	2011-12-19 11:25:40 UTC (rev 4374)
@@ -982,9 +982,6 @@
                             "302 Moved Temporarily");
 
             } else {
-                ngx_log_error(NGX_LOG_ERR, r->connection->log, 0,
-                              "upstream sent neither valid HTTP/1.0 header "
-                              "nor \"Status\" header line");
                 u->headers_in.status_n = 200;
                 ngx_str_set(&u->headers_in.status_line, "200 OK");
             }



More information about the nginx-devel mailing list