Ooops. Didn't fix the actual check but just added the comment instead. Fixed.

pull/4/head
Stefan Rubner 2012-03-14 00:29:11 +01:00
parent 07614d5abe
commit ea6323f92c
1 changed files with 3 additions and 3 deletions

View File

@ -803,7 +803,7 @@ ngx_http_fancyindex_handler(ngx_http_request_t *r)
ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0,
"http fancyindex: header subrequest status = %i",
sr->headers_out.status);
/* ngx_http_subrequest in 1.1.16 returns NGX_OK(0), not NGX_HTTP_OK(200) */
/* ngx_http_subrequest returns NGX_OK(0), not NGX_HTTP_OK(200) */
if (sr->headers_out.status != NGX_OK) {
/*
* XXX: Should we write a message to the error log just in case
@ -881,8 +881,8 @@ add_builtin_header:
"http fancyindex: header subrequest status = %i",
sr->headers_out.status);
/* see above: ngx_http_subrequest in 1.1.16 resturns NGX_OK (0) not NGX_HTTP_OK (200) */
if (sr->headers_out.status != NGX_HTTP_OK) {
/* see above: ngx_http_subrequest resturns NGX_OK (0) not NGX_HTTP_OK (200) */
if (sr->headers_out.status != NGX_OK) {
/*
* XXX: Should we write a message to the error log just in case
* we get something different from a 404?