diff options
| author | Igor Sysoev <igor@sysoev.ru> | 2003-10-12 16:49:16 +0000 |
|---|---|---|
| committer | Igor Sysoev <igor@sysoev.ru> | 2003-10-12 16:49:16 +0000 |
| commit | 0a280a3fc081877bc45cbdc0511a8cc42675213c (patch) | |
| tree | b5ea17e3be5eab555cf40f6304338a88bd641527 /src/http | |
| parent | b3e73d869ecef9248468e0ccc2d7d0abdbee495c (diff) | |
| download | nginx-0a280a3fc081877bc45cbdc0511a8cc42675213c.tar.gz nginx-0a280a3fc081877bc45cbdc0511a8cc42675213c.tar.bz2 | |
nginx-0.0.1-2003-10-12-20:49:16 import
Diffstat (limited to '')
| -rw-r--r-- | src/http/modules/ngx_http_gzip_filter.c | 8 | ||||
| -rw-r--r-- | src/http/modules/ngx_http_index_handler.c | 156 | ||||
| -rw-r--r-- | src/http/modules/proxy/ngx_http_proxy_handler.c | 29 | ||||
| -rw-r--r-- | src/http/ngx_http.c | 6 | ||||
| -rw-r--r-- | src/http/ngx_http_core_module.c | 110 | ||||
| -rw-r--r-- | src/http/ngx_http_header_filter.c | 5 | ||||
| -rw-r--r-- | src/http/ngx_http_request.c | 419 | ||||
| -rw-r--r-- | src/http/ngx_http_request.h | 5 | ||||
| -rw-r--r-- | src/http/ngx_http_special_response.c | 18 | ||||
| -rw-r--r-- | src/http/ngx_http_write_filter.c | 7 |
10 files changed, 332 insertions, 431 deletions
diff --git a/src/http/modules/ngx_http_gzip_filter.c b/src/http/modules/ngx_http_gzip_filter.c index 3e80c0b13..f5e83842a 100644 --- a/src/http/modules/ngx_http_gzip_filter.c +++ b/src/http/modules/ngx_http_gzip_filter.c @@ -23,7 +23,7 @@ typedef struct { ngx_hunk_t *out_hunk; int hunks; - int length; + off_t length; void *alloc; unsigned flush:4; @@ -90,7 +90,9 @@ ngx_module_t ngx_http_gzip_filter_module = { }; -static char gzheader[10] = { 0x1f, 0x8b, Z_DEFLATED, 0, 0, 0, 0, 0, 0, 3 }; +static char gzheader[10] = { 0x1f, + (char) 0x8b, /* suppress MSVC warning */ + Z_DEFLATED, 0, 0, 0, 0, 0, 0, 3 }; #if (HAVE_LITTLE_ENDIAN) @@ -197,7 +199,7 @@ static int ngx_http_gzip_body_filter(ngx_http_request_t *r, ngx_chain_t *in) #if 0 ngx_test_null(ctx->alloc, ngx_alloc(200K, r->log), NGX_ERROR); #else - ctx->alloc = (void *) ~NULL; + ctx->alloc = (void *) -1; #endif rc = deflateInit2(&ctx->zstream, /**/ 1, Z_DEFLATED, diff --git a/src/http/modules/ngx_http_index_handler.c b/src/http/modules/ngx_http_index_handler.c index 7c348e2ad..1bf2bd99d 100644 --- a/src/http/modules/ngx_http_index_handler.c +++ b/src/http/modules/ngx_http_index_handler.c @@ -10,10 +10,20 @@ typedef struct { } ngx_http_index_conf_t; +typedef struct { + int index; + unsigned tested:1; +} ngx_http_index_ctx_t; + + #define NGX_HTTP_DEFAULT_INDEX "index.html" -static int ngx_http_index_test_dir(ngx_http_request_t *r); +static int ngx_http_index_test_dir(ngx_http_request_t *r, + ngx_http_index_ctx_t *ctx); +static int ngx_http_index_error(ngx_http_request_t *r, + ngx_http_index_ctx_t *ctx, ngx_err_t err); + static int ngx_http_index_init(ngx_cycle_t *cycle); static void *ngx_http_index_create_conf(ngx_conf_t *cf); static char *ngx_http_index_merge_conf(ngx_conf_t *cf, @@ -58,14 +68,14 @@ ngx_module_t ngx_http_index_module = { /* - Try to open the first index file before the directory existence test - because the valid requests should be many more than invalid ones. - If open() failed then stat() should be more quickly because some data - is already cached in the kernel. - Besides Win32 has ERROR_PATH_NOT_FOUND (NGX_ENOTDIR). - Unix has ENOTDIR error, although it less helpfull - it shows only - that path contains the usual file in place of the directory. -*/ + * Try to open the first index file before the test of the directory existence + * because the valid requests should be many more than invalid ones. + * If open() failed then stat() should be more quickly because some data + * is already cached in the kernel. + * Besides Win32 has ERROR_PATH_NOT_FOUND (NGX_ENOTDIR). + * Unix has ENOTDIR error, although it less helpfull - it shows only + * that path contains the usual file in place of the directory. + */ int ngx_http_index_handler(ngx_http_request_t *r) { @@ -74,38 +84,58 @@ int ngx_http_index_handler(ngx_http_request_t *r) ngx_str_t redirect, *index; ngx_err_t err; ngx_fd_t fd; + ngx_http_index_ctx_t *ctx; ngx_http_index_conf_t *icf; ngx_http_core_loc_conf_t *clcf; - icf = ngx_http_get_module_loc_conf(r, ngx_http_index_module); - clcf = ngx_http_get_module_loc_conf(r, ngx_http_core_module); + if (r->uri.data[r->uri.len - 1] != '/') { + return NGX_DECLINED; + } - ngx_test_null(r->path.data, - ngx_palloc(r->pool, - clcf->doc_root.len + r->uri.len - + icf->max_index_len), - NGX_HTTP_INTERNAL_SERVER_ERROR); + ctx = ngx_http_get_module_ctx(r, ngx_http_index_module); + if (ctx == NULL) { + ngx_http_create_ctx(r, ctx, ngx_http_index_module, + sizeof(ngx_http_index_ctx_t), + NGX_HTTP_INTERNAL_SERVER_ERROR); + } - redirect.data = ngx_cpymem(r->path.data, clcf->doc_root.data, - clcf->doc_root.len); - file = ngx_cpystrn(redirect.data, r->uri.data, r->uri.len + 1); - r->path.len = file - r->path.data; + icf = ngx_http_get_module_loc_conf(r, ngx_http_index_module); + clcf = ngx_http_get_module_loc_conf(r, ngx_http_core_module); - test_dir = 1; - path_not_found = 1; + if (r->path.data == NULL) { + r->path_allocated = clcf->doc_root.len + r->uri.len + + icf->max_index_len; + ngx_test_null(r->path.data, + ngx_palloc(r->pool, r->path_allocated), + NGX_HTTP_INTERNAL_SERVER_ERROR); + + redirect.data = ngx_cpymem(r->path.data, clcf->doc_root.data, + clcf->doc_root.len); + file = ngx_cpystrn(redirect.data, r->uri.data, r->uri.len + 1); + r->path.len = file - r->path.data; + + } else{ + redirect.data = r->path.data + r->path.len; + file = redirect.data + r->uri.len; + } index = icf->indices.elts; - for (i = 0; i < icf->indices.nelts; i++) { + for (/* void */; ctx->index < icf->indices.nelts; ctx->index++) { - if (index[i].data[0] != '/') { - ngx_memcpy(file, index[i].data, index[i].len + 1); - name = r->path.data; + if (index[ctx->index].data[0] == '/') { + name = index[ctx->index].data; } else { - name = index[i].data; + ngx_memcpy(file, index[ctx->index].data, index[ctx->index].len + 1); + name = r->path.data; } fd = ngx_open_file(name, NGX_FILE_RDONLY, NGX_FILE_OPEN); + + if (fd == NGX_AGAIN) { + return NGX_AGAIN; + } + if (fd == NGX_INVALID_FILE) { err = ngx_errno; @@ -113,25 +143,20 @@ ngx_log_error(NGX_LOG_DEBUG, r->connection->log, err, "DEBUG: " ngx_open_file_n " %s failed", name); if (err == NGX_ENOTDIR) { - path_not_found = 1; + return ngx_http_index_error(r, ctx, err); } else if (err == NGX_EACCES) { - r->path_err = err; - return NGX_HTTP_FORBIDDEN; + return ngx_http_index_error(r, ctx, err); } - if (test_dir) { - if (path_not_found) { - r->path_err = err; - return NGX_HTTP_NOT_FOUND; - } + if (!ctx->tested) { + rc = ngx_http_index_test_dir(r, ctx); - rc = ngx_http_index_test_dir(r); if (rc != NGX_OK) { return rc; } - test_dir = 0; + ctx->tested = 1; } if (err == NGX_ENOENT) { @@ -147,14 +172,15 @@ ngx_log_error(NGX_LOG_DEBUG, r->connection->log, err, r->file.name.data = name; r->file.fd = fd; - if (index[i].data[0] == '/') { - r->file.name.len = index[i].len; - redirect.len = index[i].len; - redirect.data = index[i].data; + if (index[ctx->index].data[0] == '/') { + r->file.name.len = index[ctx->index].len; + redirect.len = index[ctx->index].len; + redirect.data = index[ctx->index].data; } else { - redirect.len = r->uri.len + index[i].len; - r->file.name.len = clcf->doc_root.len + r->uri.len + index[i].len; + redirect.len = r->uri.len + index[ctx->index].len; + r->file.name.len = clcf->doc_root.len + r->uri.len + + index[ctx->index].len; } return ngx_http_internal_redirect(r, &redirect, NULL); @@ -164,29 +190,26 @@ ngx_log_error(NGX_LOG_DEBUG, r->connection->log, err, } -static int ngx_http_index_test_dir(ngx_http_request_t *r) +static int ngx_http_index_test_dir(ngx_http_request_t *r, + ngx_http_index_ctx_t *ctx) { + ngx_err_t err; + r->path.data[r->path.len - 1] = '\0'; r->path.data[r->path.len] = '\0'; ngx_log_debug(r->connection->log, "IS_DIR: %s" _ r->path.data); -#if 0 - if (r->path_err == NGX_EACCES) { - return NGX_HTTP_FORBIDDEN; - } -#endif - if (ngx_file_type(r->path.data, &r->file.info) == -1) { - r->path_err = ngx_errno; + err = ngx_errno; - if (r->path_err == NGX_ENOENT) { + if (err == NGX_ENOENT) { r->path.data[r->path.len - 1] = '/'; - return NGX_HTTP_NOT_FOUND; + return ngx_http_index_error(r, ctx, err); } - ngx_log_error(NGX_LOG_CRIT, r->connection->log, r->path_err, + ngx_log_error(NGX_LOG_CRIT, r->connection->log, err, ngx_file_type_n " %s failed", r->path.data); return NGX_HTTP_INTERNAL_SERVER_ERROR; @@ -196,10 +219,26 @@ ngx_log_debug(r->connection->log, "IS_DIR: %s" _ r->path.data); if (ngx_is_dir(r->file.info)) { return NGX_OK; + } - } else { - return NGX_HTTP_NOT_FOUND; + /* THINK: not reached ??? */ + return ngx_http_index_error(r, ctx, 0); +} + + +static int ngx_http_index_error(ngx_http_request_t *r, + ngx_http_index_ctx_t *ctx, ngx_err_t err) +{ + if (err == NGX_EACCES) { + ngx_log_error(NGX_LOG_ERR, r->connection->log, err, + "\"%s\" is forbidden", r->path.data); + + return NGX_HTTP_FORBIDDEN; } + + ngx_log_error(NGX_LOG_ERR, r->connection->log, err, + "\"%s\" is not found", r->path.data); + return NGX_HTTP_NOT_FOUND; } @@ -212,8 +251,9 @@ static int ngx_http_index_init(ngx_cycle_t *cycle) ctx = (ngx_http_conf_ctx_t *) cycle->conf_ctx[ngx_http_module.index]; cmcf = ctx->main_conf[ngx_http_core_module.ctx_index]; - ngx_test_null(h, ngx_push_array(&cmcf->index_handlers), NGX_ERROR); - + ngx_test_null(h, ngx_push_array( + &cmcf->phases[NGX_HTTP_TRANSLATE_PHASE].handlers), + NGX_ERROR); *h = ngx_http_index_handler; return NGX_OK; diff --git a/src/http/modules/proxy/ngx_http_proxy_handler.c b/src/http/modules/proxy/ngx_http_proxy_handler.c index adb9be5c4..b21eac47a 100644 --- a/src/http/modules/proxy/ngx_http_proxy_handler.c +++ b/src/http/modules/proxy/ngx_http_proxy_handler.c @@ -292,10 +292,8 @@ static void ngx_http_proxy_send_request(ngx_http_proxy_ctx_t *p) if (chain) { ngx_add_timer(c->write, p->lcf->send_timeout); - c->write->timer_set = 1; } else { - c->write->timer_set = 0; /* TODO: del event */ } @@ -376,7 +374,6 @@ static void ngx_http_proxy_send_request(ngx_http_proxy_ctx_t *p) /* rc == NGX_AGAIN */ ngx_add_timer(c->write, p->lcf->connect_timeout); - c->write->timer_set = 1; return; } @@ -600,29 +597,11 @@ static ssize_t ngx_http_proxy_read_upstream_header(ngx_http_proxy_ctx_t *p) p->header_in->end - p->header_in->last); if (n == NGX_AGAIN) { - if (rev->timer_set) { - ngx_del_timer(rev); - } else { - rev->timer_set = 1; - } - ngx_add_timer(rev, p->lcf->read_timeout); - if (!rev->active) { - if (ngx_event_flags & NGX_HAVE_CLEAR_EVENT) { - /* kqueue */ - event = NGX_CLEAR_EVENT; - - } else { - /* select, poll, /dev/poll */ - event = NGX_LEVEL_EVENT; - } - - if (ngx_add_event(rev, NGX_READ_EVENT, event) == NGX_ERROR) { - ngx_http_proxy_finalize_request(p, - NGX_HTTP_INTERNAL_SERVER_ERROR); - return NGX_ERROR; - } + if (ngx_handle_read_event(rev) == NGX_ERROR) { + ngx_http_proxy_finalize_request(p, NGX_HTTP_INTERNAL_SERVER_ERROR); + return NGX_ERROR; } return NGX_AGAIN; @@ -1052,12 +1031,10 @@ static void ngx_http_proxy_close_connection(ngx_connection_t *c) if (c->read->timer_set) { ngx_del_timer(c->read); - c->read->timer_set = 0; } if (c->write->timer_set) { ngx_del_timer(c->write); - c->write->timer_set = 0; } /* TODO: move connection to the connection pool */ diff --git a/src/http/ngx_http.c b/src/http/ngx_http.c index 48980bbbb..c2e917815 100644 --- a/src/http/ngx_http.c +++ b/src/http/ngx_http.c @@ -201,7 +201,7 @@ static char *ngx_http_block(ngx_conf_t *cf, ngx_command_t *cmd, void *conf) } - /* init list of the handlers */ + /* init lists of the handlers */ ngx_init_array(cmcf->phases[NGX_HTTP_REWRITE_PHASE].handlers, cf->cycle->pool, 10, sizeof(ngx_http_handler_pt), @@ -219,10 +219,6 @@ static char *ngx_http_block(ngx_conf_t *cf, ngx_command_t *cmd, void *conf) cmcf->phases[NGX_HTTP_TRANSLATE_PHASE].type = NGX_OK; - ngx_init_array(cmcf->index_handlers, cf->cycle->pool, - 3, sizeof(ngx_http_handler_pt), NGX_CONF_ERROR); - - /* create the lists of the ports, the addresses and the server names to allow quickly find the server core module configuration at run-time */ diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c index 69c741c87..59d4d6f58 100644 --- a/src/http/ngx_http_core_module.c +++ b/src/http/ngx_http_core_module.c @@ -8,12 +8,11 @@ /* STUB */ int ngx_http_static_handler(ngx_http_request_t *r); +/**/ static void ngx_http_phase_event_handler(ngx_event_t *rev); static void ngx_http_run_phases(ngx_http_request_t *r); -static int ngx_http_core_index_handler(ngx_http_request_t *r); - static void *ngx_http_core_create_main_conf(ngx_conf_t *cf); static char *ngx_http_core_init_main_conf(ngx_conf_t *cf, void *conf); static void *ngx_http_core_create_srv_conf(ngx_conf_t *cf); @@ -221,9 +220,6 @@ void ngx_http_handler(ngx_http_request_t *r) r->connection->write->event_handler = ngx_http_phase_event_handler; - r->phase = 0; - r->phase_handler = 0; - ngx_http_run_phases(r); return; @@ -238,6 +234,8 @@ static void ngx_http_phase_event_handler(ngx_event_t *ev) c = ev->data; r = c->data; + ngx_log_debug(ev->log, "phase event handler"); + ngx_http_run_phases(r); return; @@ -263,6 +261,10 @@ static void ngx_http_run_phases(ngx_http_request_t *r) { rc = h[r->phase_handler](r); + if (r->closed) { + return; + } + if (rc == NGX_DECLINED) { continue; } @@ -370,8 +372,24 @@ int ngx_http_core_translate_handler(ngx_http_request_t *r) cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module); if (r->uri.data[r->uri.len - 1] == '/') { - r->content_handler = ngx_http_core_index_handler; - return NGX_OK; + if (r->path.data == NULL) { + ngx_test_null(r->path.data, + ngx_palloc(r->pool, + clcf->doc_root.len + r->uri.len), + NGX_HTTP_INTERNAL_SERVER_ERROR); + + ngx_cpystrn(ngx_cpymem(r->path.data, clcf->doc_root.data, + clcf->doc_root.len), + r->uri.data, r->uri.len + 1); + + } else { + r->path.data[r->path.len] = '\0'; + } + + ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, + "directory index of \"%s\" is forbidden", r->path.data); + + return NGX_HTTP_FORBIDDEN; } /* "+ 2" is for trailing '/' in redirect and '\0' */ @@ -388,9 +406,11 @@ ngx_log_debug(r->connection->log, "HTTP filename: '%s'" _ r->file.name.data); #if (WIN9X) - /* There is no way to open a file or a directory in Win9X with - one syscall: Win9X has no FILE_FLAG_BACKUP_SEMANTICS flag. - so we need to check its type before the opening */ + /* + * There is no way to open a file or a directory in Win9X with + * one syscall: Win9X has no FILE_FLAG_BACKUP_SEMANTICS flag. + * so we need to check its type before the opening + */ r->file.info.dwFileAttributes = GetFileAttributes(r->file.name.data); if (r->file.info.dwFileAttributes == INVALID_FILE_ATTRIBUTES) { @@ -488,42 +508,6 @@ ngx_log_debug(r->connection->log, "HTTP DIR: '%s'" _ r->file.name.data); } -static int ngx_http_core_index_handler(ngx_http_request_t *r) -{ - int i, rc; - ngx_http_handler_pt *h; - ngx_http_core_main_conf_t *cmcf; - - cmcf = ngx_http_get_module_main_conf(r, ngx_http_core_module); - - h = cmcf->index_handlers.elts; - for (i = cmcf->index_handlers.nelts; i > 0; /* void */) { - rc = h[--i](r); - - if (rc != NGX_DECLINED) { - - if (rc == NGX_HTTP_NOT_FOUND) { - ngx_log_error(NGX_LOG_ERR, r->connection->log, r->path_err, - "\"%s\" is not found", r->path.data); - } - - if (rc == NGX_HTTP_FORBIDDEN) { - ngx_log_error(NGX_LOG_ERR, r->connection->log, r->path_err, - "\"%s\" is forbidden", r->path.data); - } - - return rc; - } - } - - r->path.data[r->path.len] = '\0'; - ngx_log_error(NGX_LOG_ERR, r->connection->log, 0, - "directory index of \"%s\" is forbidden", r->path.data); - - return NGX_HTTP_FORBIDDEN; -} - - int ngx_http_send_header(ngx_http_request_t *r) { return (*ngx_http_top_header_filter)(r); @@ -591,12 +575,38 @@ int ngx_http_internal_redirect(ngx_http_request_t *r, } } + /* clear the modules contexts */ + ngx_memzero(r->ctx, sizeof(void *) * ngx_http_max_module); + + r->phase = 0; + r->phase_handler = 0; + ngx_http_handler(r); return NGX_OK; } +#if 1 /* STUB: test the delay http handler */ + +int ngx_http_delay_handler(ngx_http_request_t *r) +{ + static int on; + + if (on++ == 0) { + ngx_log_debug(r->connection->log, "SET http delay"); + ngx_add_timer(r->connection->write, 10000); + return NGX_AGAIN; + } + + r->connection->write->timedout = 0; + ngx_log_debug(r->connection->log, "RESET http delay"); + return NGX_DECLINED; +} + +#endif + + static int ngx_http_core_init(ngx_cycle_t *cycle) { ngx_http_handler_pt *h; @@ -609,9 +619,15 @@ static int ngx_http_core_init(ngx_cycle_t *cycle) ngx_test_null(h, ngx_push_array( &cmcf->phases[NGX_HTTP_TRANSLATE_PHASE].handlers), NGX_ERROR); - *h = ngx_http_core_translate_handler; +#if 0 + ngx_test_null(h, ngx_push_array( + &cmcf->phases[NGX_HTTP_TRANSLATE_PHASE].handlers), + NGX_ERROR); + *h = ngx_http_delay_handler; +#endif + return NGX_OK; } diff --git a/src/http/ngx_http_header_filter.c b/src/http/ngx_http_header_filter.c index 346c33226..b05452ae7 100644 --- a/src/http/ngx_http_header_filter.c +++ b/src/http/ngx_http_header_filter.c @@ -121,7 +121,10 @@ static int ngx_http_header_filter(ngx_http_request_t *r) } else if (r->headers_out.status < NGX_HTTP_BAD_REQUEST) { /* 3XX */ status = r->headers_out.status - NGX_HTTP_MOVED_PERMANENTLY + 8; - r->header_only = 1; + + if (r->headers_out.status == NGX_HTTP_NOT_MODIFIED) { + r->header_only = 1; + } } else if (r->headers_out.status < NGX_HTTP_INTERNAL_SERVER_ERROR) { /* 4XX */ diff --git a/src/http/ngx_http_request.c b/src/http/ngx_http_request.c index cde7f1d64..5efbf3e2d 100644 --- a/src/http/ngx_http_request.c +++ b/src/http/ngx_http_request.c @@ -22,7 +22,8 @@ static void ngx_http_set_lingering_close(ngx_http_request_t *r); static void ngx_http_lingering_close_handler(ngx_event_t *ev); static void ngx_http_empty_handler(ngx_event_t *wev); -static void ngx_http_header_parse_error(ngx_http_request_t *r, int parse_err); +static void ngx_http_header_parse_error(ngx_http_request_t *r, + int parse_err, int error); static size_t ngx_http_log_error(void *data, char *buf, size_t len); @@ -65,7 +66,6 @@ static ngx_http_header_t headers_in[] = { void ngx_http_init_connection(ngx_connection_t *c) { - int event; ngx_event_t *rev; ngx_http_log_ctx_t *lctx; @@ -104,9 +104,8 @@ void ngx_http_init_connection(ngx_connection_t *c) } ngx_add_timer(rev, c->listening->post_accept_timeout); - rev->timer_set = 1; - if (ngx_event_flags & (NGX_HAVE_AIO_EVENT|NGX_HAVE_EDGE_EVENT)) { + if (ngx_event_flags & (NGX_USE_AIO_EVENT|NGX_USE_EDGE_EVENT)) { /* aio, iocp, epoll */ ngx_http_init_request(rev); return; @@ -116,25 +115,6 @@ void ngx_http_init_connection(ngx_connection_t *c) ngx_http_close_connection(c); return; } - -#if 0 - - if (ngx_event_flags & NGX_HAVE_CLEAR_EVENT) { - /* kqueue */ - event = NGX_CLEAR_EVENT; - - } else { - /* select, poll, /dev/poll */ - event = NGX_LEVEL_EVENT; - } - - if (ngx_add_event(rev, NGX_READ_EVENT, event) == NGX_ERROR) { - ngx_http_close_connection(c); - } - -#endif - - return; } @@ -153,10 +133,16 @@ static void ngx_http_init_request(ngx_event_t *rev) c = rev->data; - r = ngx_pcalloc(c->pool, sizeof(ngx_http_request_t)); - if (r == NULL) { - ngx_http_close_connection(c); - return; + if (c->data) { + r = c->data; + ngx_memzero(r, sizeof(ngx_http_request_t)); + + } else { + r = ngx_pcalloc(c->pool, sizeof(ngx_http_request_t)); + if (r == NULL) { + ngx_http_close_connection(c); + return; + } } /* find the server configuration for the address:port */ @@ -315,8 +301,8 @@ static void ngx_http_process_request_line(ngx_event_t *rev) r->request_line.data = r->request_start; r->request_line.data[r->request_line.len] = '\0'; - ngx_http_header_parse_error(r, NGX_HTTP_PARSE_INVALID_REQUEST); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + ngx_http_header_parse_error(r, NGX_HTTP_PARSE_INVALID_REQUEST, + NGX_HTTP_BAD_REQUEST); return; } @@ -328,8 +314,8 @@ static void ngx_http_process_request_line(ngx_event_t *rev) { /* no space for "\r\n" at the end of the header */ - ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI); - ngx_http_finalize_request(r, NGX_HTTP_REQUEST_URI_TOO_LARGE); + ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI, + NGX_HTTP_REQUEST_URI_TOO_LARGE); return; } @@ -459,8 +445,7 @@ static void ngx_http_process_request_line(ngx_event_t *rev) /* there was error while a request line parsing */ - ngx_http_header_parse_error(r, rc); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + ngx_http_header_parse_error(r, rc, NGX_HTTP_BAD_REQUEST); return; } @@ -469,11 +454,13 @@ static void ngx_http_process_request_line(ngx_event_t *rev) if (r->header_in->last == r->header_in->end) { - /* If it's a pipelined request and a request line is not complete - then we need to copy it to the start of the r->header_in hunk. - We need to copy it here only if the large client headers - are enabled otherwise a request line had been already copied - to the start of the r->header_in hunk in ngx_http_set_keepalive() */ + /* + * If it's a pipelined request and a request line is not complete + * then we need to copy it to the start of the r->header_in hunk. + * We need to copy it here only if the large client headers + * are enabled otherwise a request line had been already copied + * to the start of the r->header_in hunk in ngx_http_set_keepalive(). + */ cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module); @@ -481,9 +468,8 @@ static void ngx_http_process_request_line(ngx_event_t *rev) offset = r->request_start - r->header_in->start; if (offset == 0) { - ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI); - ngx_http_finalize_request(r, NGX_HTTP_REQUEST_URI_TOO_LARGE); - + ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI, + NGX_HTTP_REQUEST_URI_TOO_LARGE); return; } @@ -504,8 +490,8 @@ static void ngx_http_process_request_line(ngx_event_t *rev) } } else { - ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI); - ngx_http_finalize_request(r, NGX_HTTP_REQUEST_URI_TOO_LARGE); + ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_URI, + NGX_HTTP_REQUEST_URI_TOO_LARGE); } } @@ -651,8 +637,8 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) } else { if (r->http_version > NGX_HTTP_VERSION_10) { ngx_http_header_parse_error(r, - NGX_HTTP_PARSE_NO_HOST_HEADER); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + NGX_HTTP_PARSE_NO_HOST_HEADER, + NGX_HTTP_BAD_REQUEST); return; } r->headers_in.host_name_len = 0; @@ -662,16 +648,20 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) r->headers_in.content_length_n = ngx_atoi(r->headers_in.content_length->value.data, r->headers_in.content_length->value.len); + if (r->headers_in.content_length_n == NGX_ERROR) { ngx_http_header_parse_error(r, - NGX_HTTP_PARSE_INVALID_CL_HEADER); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + NGX_HTTP_PARSE_INVALID_CL_HEADER, + NGX_HTTP_BAD_REQUEST); return; } } + if (r->header_timeout_set) { + ngx_del_timer(rev); + } + rev->event_handler = ngx_http_block_read; - c->write->event_handler = ngx_http_writer; ngx_http_handler(r); return; @@ -679,9 +669,7 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) /* there was error while a header line parsing */ - ngx_http_header_parse_error(r, rc); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); - + ngx_http_header_parse_error(r, rc, NGX_HTTP_BAD_REQUEST); return; } @@ -697,8 +685,8 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) if (offset == 0) { ngx_http_header_parse_error(r, - NGX_HTTP_PARSE_TOO_LONG_HEADER); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + NGX_HTTP_PARSE_TOO_LONG_HEADER, + NGX_HTTP_BAD_REQUEST); return; } @@ -713,8 +701,8 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) r->header_end -= offset; } else { - ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_HEADER); - ngx_http_finalize_request(r, NGX_HTTP_BAD_REQUEST); + ngx_http_header_parse_error(r, NGX_HTTP_PARSE_TOO_LONG_HEADER, + NGX_HTTP_BAD_REQUEST); return; } } @@ -724,7 +712,6 @@ static void ngx_http_process_request_headers(ngx_event_t *rev) static ssize_t ngx_http_read_request_header(ngx_http_request_t *r) { - int event; ssize_t n; ngx_event_t *rev; ngx_http_core_srv_conf_t *cscf; @@ -743,14 +730,7 @@ static ssize_t ngx_http_read_request_header(ngx_http_request_t *r) if (n == NGX_AGAIN) { if (!r->header_timeout_set) { - if (rev->timer_set) { - ngx_del_timer(rev); - } else { - rev->timer_set = 1; - } - cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module); - ngx_add_timer(rev, cscf->client_header_timeout); r->header_timeout_set = 1; } @@ -761,25 +741,6 @@ static ssize_t ngx_http_read_request_header(ngx_http_request_t *r) return NGX_ERROR; } -#if 0 - if (!rev->active) { - if (ngx_event_flags & NGX_HAVE_CLEAR_EVENT) { - /* kqueue */ - event = NGX_CLEAR_EVENT; - - } else { - /* select, poll, /dev/poll */ - event = NGX_LEVEL_EVENT; - } - - if (ngx_add_event(rev, NGX_READ_EVENT, event) == NGX_ERROR) { - ngx_http_close_request(r, NGX_HTTP_INTERNAL_SERVER_ERROR); - ngx_http_close_connection(r->connection); - return NGX_ERROR; - } - } -#endif - return NGX_AGAIN; } @@ -802,7 +763,7 @@ static ssize_t ngx_http_read_request_header(ngx_http_request_t *r) void ngx_http_finalize_request(ngx_http_request_t *r, int rc) { - ngx_event_t *rev, *wev; + ngx_log_debug(r->connection->log, "finalize http request"); if (r->main || r->closed) { return; @@ -810,16 +771,12 @@ void ngx_http_finalize_request(ngx_http_request_t *r, int rc) if (rc >= NGX_HTTP_SPECIAL_RESPONSE) { - rev = r->connection->read; - if (rev->timer_set) { - ngx_del_timer(rev); - rev->timer_set = 0; + if (r->connection->read->timer_set) { + ngx_del_timer(r->connection->read); } - wev = r->connection->write; - if (wev->timer_set) { - ngx_del_timer(wev); - wev->timer_set = 0; + if (r->connection->write->timer_set) { + ngx_del_timer(r->connection->write); } ngx_http_finalize_request(r, ngx_http_special_response_handler(r, rc)); @@ -836,16 +793,12 @@ void ngx_http_finalize_request(ngx_http_request_t *r, int rc) return; } - rev = r->connection->read; - if (rev->timer_set) { - ngx_del_timer(rev); - rev->timer_set = 0; + if (r->connection->read->timer_set) { + ngx_del_timer(r->connection->read); } - wev = r->connection->write; - if (wev->timer_set) { - ngx_del_timer(wev); - wev->timer_set = 0; + if (r->connection->write->timer_set) { + ngx_del_timer(r->connection->write); } if (r->keepalive != 0) { @@ -865,7 +818,6 @@ void ngx_http_finalize_request(ngx_http_request_t *r, int rc) static void ngx_http_set_write_handler(ngx_http_request_t *r) { - int event; ngx_event_t *wev; ngx_http_core_loc_conf_t *clcf; @@ -879,44 +831,12 @@ static void ngx_http_set_write_handler(ngx_http_request_t *r) clcf = ngx_http_get_module_loc_conf(r->main ? r->main : r, ngx_http_core_module); ngx_add_timer(wev, clcf->send_timeout); - wev->timer_set = 1; if (ngx_handle_write_event(wev, clcf->send_lowat) == NGX_ERROR) { ngx_http_close_request(r, 0); ngx_http_close_connection(r->connection); } -#if 0 - - if (ngx_event_flags & (NGX_HAVE_AIO_EVENT|NGX_HAVE_EDGE_EVENT)) { - /* aio, iocp, epoll */ - return; - } - -#if (HAVE_LOWAT_EVENT) /* kqueue's NOTE_LOWAT */ - - if (ngx_event_flags & NGX_HAVE_LOWAT_EVENT) { - wev->lowat = clcf->send_lowat; - } - -#endif - - if (ngx_event_flags & NGX_HAVE_CLEAR_EVENT) { - /* kqueue */ - event = NGX_CLEAR_EVENT; - - } else { - /* select, poll, /dev/poll */ - event = NGX_LEVEL_EVENT; - } - - if (ngx_add_event(wev, NGX_WRITE_EVENT, event) == NGX_ERROR) { - ngx_http_close_request(r, 0); - ngx_http_close_connection(r->connection); - } - -#endif - return; } @@ -924,7 +844,6 @@ static void ngx_http_set_write_handler(ngx_http_request_t *r) void ngx_http_writer(ngx_event_t *wev) { int rc; - ngx_event_t *rev; ngx_connection_t *c; ngx_http_request_t *r; ngx_http_core_loc_conf_t *clcf; @@ -937,53 +856,23 @@ void ngx_http_writer(ngx_event_t *wev) ngx_log_debug(c->log, "writer output filter: %d" _ rc); if (rc == NGX_AGAIN) { - - clcf = ngx_http_get_module_loc_conf(r->main ? r->main : r, - ngx_http_core_module); - if (wev->timer_set) { - ngx_del_timer(wev); - } else { - wev->timer_set = 1; + if (!wev->ready) { + clcf = ngx_http_get_module_loc_conf(r->main ? r->main : r, + ngx_http_core_module); + ngx_add_timer(wev, clcf->send_timeout); } - ngx_add_timer(wev, clcf->send_timeout); - - return; - } + if (ngx_handle_level_write_event(wev) == NGX_ERROR) { + ngx_http_close_request(r, 0); + ngx_http_close_connection(r->connection); + } - if (rc == NGX_ERROR) { - ngx_http_close_request(r, 0); - ngx_http_close_connection(c); return; } - /* rc == NGX_OK */ - ngx_log_debug(c->log, "http writer done"); - rev = r->connection->read; - if (rev->timer_set) { - ngx_del_timer(rev); - rev->timer_set = 0; - } - - if (wev->timer_set) { - ngx_del_timer(wev); - wev->timer_set = 0; - } - - if (r->keepalive != 0) { - ngx_http_set_keepalive(r); - - } else if (r->lingering_close) { - ngx_http_set_lingering_close(r); - - } else { - ngx_http_close_request(r, 0); - ngx_http_close_connection(r->connection); - } - - return; + ngx_http_finalize_request(r, rc); } @@ -996,22 +885,15 @@ static void ngx_http_block_read(ngx_event_t *rev) /* aio does not call this handler */ - if (ngx_event_flags & NGX_USE_LEVEL_EVENT) { - - /* select, poll, /dev/poll */ - - rev->blocked = 1; - + if ((ngx_event_flags & NGX_USE_LEVEL_EVENT) && rev->active) { if (ngx_del_event(rev, NGX_READ_EVENT, 0) == NGX_ERROR) { - c = (ngx_connection_t *) rev->data; - r = (ngx_http_request_t *) c->data; + c = rev->data; + r = c->data; ngx_http_close_request(r, 0); ngx_http_close_connection(c); } } - /* kqueue, epoll */ - return; } @@ -1027,39 +909,33 @@ int ngx_http_discard_body(ngx_http_request_t *r) if (rev->timer_set) { ngx_del_timer(rev); - rev->timer_set = 0; } - if (r->headers_in.content_length_n > 0) { + if (r->headers_in.content_length_n <= 0) { + return NGX_OK; + } - size = r->header_in->last - r->header_in->pos; + size = r->header_in->last - r->header_in->pos; - if (size) { - if (r->headers_in.content_length_n > size) { - r->headers_in.content_length_n -= size; + if (size) { + if (r->headers_in.content_length_n > size) { + r->headers_in.content_length_n -= size; - } else { - r->header_in->pos += r->headers_in.content_length_n; - r->headers_in.content_length_n = 0; - return NGX_OK; - } + } else { + r->header_in->pos += r->headers_in.content_length_n; + r->headers_in.content_length_n = 0; + return NGX_OK; } + } - rev->event_handler = ngx_http_read_discarded_body_event; + rev->event_handler = ngx_http_read_discarded_body_event; - if (rev->blocked) { - if (ngx_event_flags & NGX_USE_LEVEL_EVENT) { - if (ngx_add_event(rev, NGX_READ_EVENT, NGX_LEVEL_EVENT) - == NGX_ERROR) { - return NGX_HTTP_INTERNAL_SERVER_ERROR; - } - } - - rev->blocked = 0; - return ngx_http_read_discarded_body(r); - } + if (ngx_handle_level_read_event(rev) == NGX_ERROR) { + return NGX_HTTP_INTERNAL_SERVER_ERROR; } + return ngx_http_read_discarded_body(r); + return NGX_OK; } @@ -1075,12 +951,18 @@ static void ngx_http_read_discarded_body_event(ngx_event_t *rev) rc = ngx_http_read_discarded_body(r); + if (rc == NGX_AGAIN) { + if (ngx_handle_level_read_event(rev) == NGX_ERROR) { + ngx_http_close_request(r, rc); + ngx_http_close_connection(c); + return; + } + } + if (rc != NGX_OK) { ngx_http_close_request(r, rc); ngx_http_close_connection(c); } - - return; } @@ -1091,6 +973,10 @@ static int ngx_http_read_discarded_body(ngx_http_request_t *r) ngx_log_debug(r->connection->log, "http read discarded body"); + if (r->headers_in.content_length_n == 0) { + return NGX_OK; + } + clcf = ngx_http_get_module_loc_conf(r, ngx_http_core_module); if (r->discarded_buffer == NULL) { @@ -1101,6 +987,7 @@ static int ngx_http_read_discarded_body(ngx_http_request_t *r) } size = r->headers_in.content_length_n; + if (size > clcf->discarded_buffer_size) { size = clcf->discarded_buffer_size; } @@ -1111,7 +998,7 @@ static int ngx_http_read_discarded_body(ngx_http_request_t *r) } if (n == NGX_AGAIN) { - return NGX_OK; + return NGX_AGAIN; } r->headers_in.content_length_n -= n; @@ -1122,7 +1009,7 @@ static int ngx_http_read_discarded_body(ngx_http_request_t *r) static void ngx_http_set_keepalive(ngx_http_request_t *r) { - int len, blocked; + int len; ngx_hunk_t *h; ngx_event_t *rev, *wev; ngx_connection_t *c; @@ -1139,39 +1026,26 @@ static void ngx_http_set_keepalive(ngx_http_request_t *r) ctx->action = "closing request"; ngx_http_close_request(r, 0); - if (rev->timer_set) { - ngx_del_timer(rev); - } else { - rev->timer_set = 1; - } - clcf = ngx_http_get_module_loc_conf(r, ngx_http_core_module); - ngx_add_timer(rev, clcf->keepalive_timeout); - if (rev->blocked && (ngx_event_flags & NGX_USE_LEVEL_EVENT)) { - if (ngx_add_event(rev, NGX_READ_EVENT, NGX_LEVEL_EVENT) == NGX_ERROR) { - ngx_http_close_connection(c); - return; - } - - blocked = 1; - rev->blocked = 0; - - } else { - blocked = 0; + if (ngx_handle_level_read_event(rev) == NGX_ERROR) { + ngx_http_close_connection(c); + return; } h = c->buffer; - /* pipelined request */ if (h->pos < h->last) { - /* We do not know here whether a pipelined request is complete - so if the large client headers are not enabled - we need to copy the data to the start of c->buffer. - This copy should be rare because clients that support - pipelined requests (Mozilla 1.x, Opera 6.x) are still rare */ + /* Pipelined request. + * + * We do not know here whether a pipelined request is complete + * so if the large client headers are not enabled + * we need to copy the data to the start of c->buffer. + * This copy should be rare because clients that support + * pipelined requests (Mozilla 1.x, Opera 6.x+) are still rare. + */ cscf = ngx_http_get_module_srv_conf(r, ngx_http_core_module); @@ -1195,16 +1069,12 @@ static void ngx_http_set_keepalive(ngx_http_request_t *r) h->pos = h->last = h->start; rev->event_handler = ngx_http_keepalive_handler; wev = c->write; + wev->event_handler = ngx_http_empty_handler; - if (wev->active) { - if (ngx_event_flags & NGX_USE_LEVEL_EVENT) { - if (ngx_del_event(wev, NGX_WRITE_EVENT, 0) == NGX_ERROR) { - ngx_http_close_connection(c); - return; - } - - } else if ((ngx_event_flags & NGX_HAVE_AIO_EVENT) == 0) { - wev->event_handler = ngx_http_empty_handler; + if ((ngx_event_flags & NGX_USE_LEVEL_EVENT) && wev->active) { + if (ngx_del_event(wev, NGX_WRITE_EVENT, 0) == NGX_ERROR) { + ngx_http_close_connection(c); + return; } } @@ -1220,7 +1090,7 @@ static void ngx_http_set_keepalive(ngx_http_request_t *r) c->tcp_nopush = 0; } - if ((ngx_event_flags & NGX_HAVE_AIO_EVENT) || blocked) { + if (rev->ready || (ngx_event_flags & NGX_USE_AIO_EVENT)) { ngx_http_keepalive_handler(rev); } } @@ -1228,7 +1098,7 @@ static void ngx_http_set_keepalive(ngx_http_request_t *r) static void ngx_http_keepalive_handler(ngx_event_t *rev) { - ssize_t n; + ssize_t n; ngx_connection_t *c; ngx_http_log_ctx_t *lctx; @@ -1241,8 +1111,10 @@ static void ngx_http_keepalive_handler(ngx_event_t *rev) return; } - /* MSIE closes a keepalive connection with RST flag - so we ignore ECONNRESET here */ + /* + * MSIE closes a keepalive connection with RST flag + * so we ignore ECONNRESET here. + */ rev->ignore_econnreset = 1; ngx_set_socket_errno(0); @@ -1278,46 +1150,35 @@ static void ngx_http_keepalive_handler(ngx_event_t *rev) static void ngx_http_set_lingering_close(ngx_http_request_t *r) { - ngx_event_t *rev; + ngx_event_t *rev, *wev; ngx_connection_t *c; ngx_http_core_loc_conf_t *clcf; c = r->connection; - rev = c->read; clcf = ngx_http_get_module_loc_conf(r, ngx_http_core_module); - r->lingering_time = ngx_time() + clcf->lingering_time / 1000; + rev = c->read; rev->event_handler = ngx_http_lingering_close_handler; - if (rev->timer_set) { - ngx_del_timer(rev); - } else { - rev->timer_set = 1; + r->lingering_time = ngx_time() + clcf->lingering_time / 1000; + ngx_add_timer(rev, clcf->lingering_timeout); + + if (ngx_handle_level_read_event(rev) == NGX_ERROR) { + ngx_http_close_request(r, 0); + ngx_http_close_connection(c); + return; } - ngx_add_timer(rev, clcf->lingering_timeout); + wev = c->write; + wev->event_handler = ngx_http_empty_handler; - if (rev->blocked && (ngx_event_flags & NGX_USE_LEVEL_EVENT)) { - if (ngx_add_event(rev, NGX_READ_EVENT, NGX_LEVEL_EVENT) == NGX_ERROR) { + if ((ngx_event_flags & NGX_USE_LEVEL_EVENT) && wev->active) { + if (ngx_del_event(wev, NGX_WRITE_EVENT, 0) == NGX_ERROR) { ngx_http_close_request(r, 0); ngx_http_close_connection(c); return; } - rev->blocked = 0; - } - - if (c->write->active) { - if (ngx_event_flags & NGX_USE_LEVEL_EVENT) { - if (ngx_del_event(c->write, NGX_WRITE_EVENT, 0) == NGX_ERROR) { - ngx_http_close_request(r, 0); - ngx_http_close_connection(c); - return; - } - - } else if ((ngx_event_flags & NGX_HAVE_AIO_EVENT) == 0) { - c->write->event_handler = ngx_http_empty_handler; - } } if (ngx_shutdown_socket(c->fd, NGX_WRITE_SHUTDOWN) == -1) { @@ -1328,7 +1189,7 @@ static void ngx_http_set_lingering_close(ngx_http_request_t *r) return; } - if (rev->ready || (ngx_event_flags & NGX_HAVE_AIO_EVENT)) { + if (rev->ready || (ngx_event_flags & NGX_USE_AIO_EVENT)) { ngx_http_lingering_close_handler(rev); } } @@ -1401,11 +1262,6 @@ static void ngx_http_lingering_close_handler(ngx_event_t *rev) timer = clcf->lingering_timeout; } - if (rev->timer_set) { - ngx_del_timer(rev); - } else { - rev->timer_set = 1; - } ngx_add_timer(rev, timer); return; @@ -1414,7 +1270,7 @@ static void ngx_http_lingering_close_handler(ngx_event_t *rev) static void ngx_http_empty_handler(ngx_event_t *wev) { - ngx_log_debug(wev->log, "http empty handler"); + ngx_log_debug(wev->log, "http EMPTY handler"); return; } @@ -1457,7 +1313,7 @@ void ngx_http_close_request(ngx_http_request_t *r, int error) } /* ctx->url was allocated from r->pool */ - ctx = (ngx_http_log_ctx_t *) r->connection->log->data; + ctx = r->connection->log->data; ctx->url = NULL; ngx_destroy_pool(r->pool); @@ -1479,12 +1335,10 @@ void ngx_http_close_connection(ngx_connection_t *c) if (c->read->timer_set) { ngx_del_timer(c->read); - c->read->timer_set = 0; } if (c->write->timer_set) { ngx_del_timer(c->write); - c->write->timer_set = 0; } if (ngx_del_conn) { @@ -1513,7 +1367,8 @@ void ngx_http_close_connection(ngx_connection_t *c) } -static void ngx_http_header_parse_error(ngx_http_request_t *r, int parse_err) +static void ngx_http_header_parse_error(ngx_http_request_t *r, + int parse_err, int error) { ngx_http_log_ctx_t *ctx; @@ -1532,6 +1387,8 @@ static void ngx_http_header_parse_error(ngx_http_request_t *r, int parse_err) } r->connection->log->handler = ngx_http_log_error; + + ngx_http_finalize_request(r, error); } diff --git a/src/http/ngx_http_request.h b/src/http/ngx_http_request.h index 1f8b254b9..c3029909f 100644 --- a/src/http/ngx_http_request.h +++ b/src/http/ngx_http_request.h @@ -157,6 +157,8 @@ struct ngx_http_request_s { ngx_str_t args; ngx_str_t exten; ngx_str_t unparsed_uri; + ngx_str_t path; + int path_allocated; ngx_http_request_t *main; @@ -172,9 +174,6 @@ struct ngx_http_request_s { char *discarded_buffer; - ngx_str_t path; - int path_err; - /* URI is not started with '/' - "GET http://" */ unsigned unusual_uri:1; /* URI with "/.", "%" and on Win32 with "//" */ diff --git a/src/http/ngx_http_special_response.c b/src/http/ngx_http_special_response.c index 825d8bc97..0e16c987d 100644 --- a/src/http/ngx_http_special_response.c +++ b/src/http/ngx_http_special_response.c @@ -22,6 +22,14 @@ static char msie_stub[] = ; +static char error_301_page[] = +"<html>" CRLF +"<head><title>301 Moved Permanently</title></head>" CRLF +"<body bgcolor=\"white\">" CRLF +"<center><h1>301 Moved Permanently</h1></center>" CRLF +; + + static char error_302_page[] = "<html>" CRLF "<head><title>302 Found</title></head>" CRLF @@ -111,8 +119,8 @@ static char error_504_page[] = static ngx_str_t error_pages[] = { - ngx_null_string, /* 300 */ - ngx_null_string, /* 301 */ + /* ngx_null_string, */ /* 300 */ + ngx_string(error_301_page), ngx_string(error_302_page), ngx_null_string, /* 303 */ @@ -224,7 +232,11 @@ int ngx_http_special_response_handler(ngx_http_request_t *r, int error) h->pos = error_tail; h->last = error_tail + sizeof(error_tail) - 1; - if (/* STUB: "msie_padding on/off" */ 1) { + if (/* STUB: "msie_padding on/off" */ 1 + && r->http_version >= NGX_HTTP_VERSION_10 + && error >= NGX_HTTP_BAD_REQUEST) + { + if (ngx_http_output_filter(r, h) == NGX_ERROR) { return NGX_ERROR; } diff --git a/src/http/ngx_http_write_filter.c b/src/http/ngx_http_write_filter.c index cf587508d..0f580a285 100644 --- a/src/http/ngx_http_write_filter.c +++ b/src/http/ngx_http_write_filter.c @@ -139,7 +139,7 @@ int ngx_http_write_filter(ngx_http_request_t *r, ngx_chain_t *in) return NGX_OK; } - if (!r->connection->write->ready || r->connection->write->delayed) { + if (r->connection->write->delayed) { return NGX_AGAIN; } @@ -161,10 +161,9 @@ int ngx_http_write_filter(ngx_http_request_t *r, ngx_chain_t *in) if (chain == NULL) { return NGX_OK; - - } else { - return NGX_AGAIN; } + + return NGX_AGAIN; } |
