Mercurial > hg > nginx-vendor-0-5
diff src/http/ngx_http_upstream.c @ 112:408f195b3482 NGINX_0_3_3
nginx 0.3.3
*) Change: the "bl" and "af" parameters of the "listen" directive was
renamed to the "backlog" and "accept_filter".
*) Feature: the "rcvbuf" and "sndbuf" parameters of the "listen"
directive.
*) Change: the "$msec" log parameter does not require now the
additional the gettimeofday() system call.
*) Feature: the -t switch now tests the "listen" directives.
*) Bugfix: if the invalid address was specified in the "listen"
directive, then after the -HUP signal nginx left an open socket in
the CLOSED state.
*) Bugfix: the mime type may be incorrectly set to default value for
index file with variable in the name; bug appeared in 0.3.0.
*) Feature: the "timer_resolution" directive.
*) Feature: the millisecond "$upstream_response_time" log parameter.
*) Bugfix: a temporary file with client request body now is removed
just after the response header was transferred to a client.
*) Bugfix: OpenSSL 0.9.6 compatibility.
*) Bugfix: the SSL certificate and key file paths could not be relative.
*) Bugfix: the "ssl_prefer_server_ciphers" directive did not work in
the ngx_imap_ssl_module.
*) Bugfix: the "ssl_protocols" directive allowed to specify the single
protocol only.
author | Igor Sysoev <http://sysoev.ru> |
---|---|
date | Wed, 19 Oct 2005 00:00:00 +0400 |
parents | dad2fe8ecf08 |
children | 644a7935144b |
line wrap: on
line diff
--- a/src/http/ngx_http_upstream.c +++ b/src/http/ngx_http_upstream.c @@ -66,6 +66,8 @@ static size_t ngx_http_upstream_log_stat uintptr_t data); static u_char *ngx_http_upstream_log_status(ngx_http_request_t *r, u_char *buf, ngx_http_log_op_t *op); +static u_char *ngx_http_upstream_log_response_time(ngx_http_request_t *r, + u_char *buf, ngx_http_log_op_t *op); static u_char *ngx_http_upstream_log_error(ngx_http_request_t *r, u_char *buf, size_t len); @@ -204,6 +206,8 @@ static ngx_http_log_op_name_t ngx_http_u { ngx_string("upstream_status"), 0, NULL, ngx_http_upstream_log_status_getlen, ngx_http_upstream_log_status }, + { ngx_string("upstream_response_time"), NGX_TIME_T_LEN + 4, NULL, NULL, + ngx_http_upstream_log_response_time }, { ngx_null_string, 0, NULL, NULL, NULL } }; @@ -217,6 +221,7 @@ char *ngx_http_upstream_header_errors[] void ngx_http_upstream_init(ngx_http_request_t *r) { + ngx_time_t *tp; ngx_connection_t *c; ngx_http_upstream_t *u; ngx_http_core_loc_conf_t *clcf; @@ -286,6 +291,10 @@ ngx_http_upstream_init(ngx_http_request_ ngx_memzero(u->state, sizeof(ngx_http_upstream_state_t)); + tp = ngx_timeofday(); + + u->state->response_time = tp->sec * 1000 + tp->msec; + ngx_http_upstream_connect(r, u); } @@ -978,6 +987,8 @@ ngx_http_upstream_send_response(ngx_http ngx_list_part_t *part; ngx_table_elt_t *h; ngx_event_pipe_t *p; + ngx_pool_cleanup_t *cl; + ngx_pool_cleanup_file_t *clf; ngx_http_core_loc_conf_t *clcf; ngx_http_upstream_header_t *hh; ngx_http_upstream_main_conf_t *umcf; @@ -1033,6 +1044,20 @@ ngx_http_upstream_send_response(ngx_http u->header_sent = 1; + if (r->request_body->temp_file) { + for (cl = r->pool->cleanup; cl; cl = cl->next) { + if (cl->handler == ngx_pool_cleanup_file) { + clf = cl->data; + + if (clf->fd == r->request_body->temp_file->file.fd) { + cl->handler(clf); + cl->handler = NULL; + break; + } + } + } + } + /* TODO: preallocate event_pipe bufs, look "Content-Length" */ #if 0 @@ -1404,9 +1429,17 @@ static void ngx_http_upstream_finalize_request(ngx_http_request_t *r, ngx_http_upstream_t *u, ngx_int_t rc) { + ngx_time_t *tp; + ngx_log_debug1(NGX_LOG_DEBUG_HTTP, r->connection->log, 0, "finalize http upstream request: %i", rc); + if (u->state->response_time) { + tp = ngx_timeofday(); + u->state->response_time = tp->sec * 1000 + tp->msec + - u->state->response_time; + } + u->finalize_request(r, rc); if (u->peer.connection) { @@ -1419,8 +1452,7 @@ ngx_http_upstream_finalize_request(ngx_h u->peer.connection = NULL; - if (u->header_sent - && (rc == NGX_ERROR || rc >= NGX_HTTP_SPECIAL_RESPONSE)) + if (u->header_sent && (rc == NGX_ERROR || rc >= NGX_HTTP_SPECIAL_RESPONSE)) { rc = 0; } @@ -1783,6 +1815,44 @@ ngx_http_upstream_log_status(ngx_http_re static u_char * +ngx_http_upstream_log_response_time(ngx_http_request_t *r, u_char *buf, + ngx_http_log_op_t *op) +{ + ngx_uint_t i; + ngx_http_upstream_t *u; + ngx_http_upstream_state_t *state; + + u = r->upstream; + + if (u == NULL) { + *buf = '-'; + return buf + 1; + } + + i = 0; + state = u->states.elts; + + for ( ;; ) { + if (state[i].status == 0) { + *buf++ = '-'; + + } else { + buf = ngx_sprintf(buf, "%d.%03d", + state[i].response_time / 1000, + state[i].response_time % 1000); + } + + if (++i == u->states.nelts) { + return buf; + } + + *buf++ = ','; + *buf++ = ' '; + } +} + + +static u_char * ngx_http_upstream_log_error(ngx_http_request_t *r, u_char *buf, size_t len) { u_char *p;