diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2011-12-19 11:25:40 +0000 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2011-12-19 11:25:40 +0000 |
| commit | 8a85ab8e2026bbe930dcb2482c739dc2a078a906 (patch) | |
| tree | 52913a4339d5d4ed2422c82dd837723d0f6c0b85 | |
| parent | 32e2038026f5ccb7d1c86dbbad041b949a7de654 (diff) | |
| download | nginx-8a85ab8e2026bbe930dcb2482c739dc2a078a906.tar.gz nginx-8a85ab8e2026bbe930dcb2482c739dc2a078a906.zip | |
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".
| -rw-r--r-- | src/http/modules/ngx_http_scgi_module.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/http/modules/ngx_http_scgi_module.c b/src/http/modules/ngx_http_scgi_module.c index 97cf1aafb..92e528528 100644 --- a/src/http/modules/ngx_http_scgi_module.c +++ b/src/http/modules/ngx_http_scgi_module.c @@ -982,9 +982,6 @@ ngx_http_scgi_process_header(ngx_http_request_t *r) "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"); } |