mirror of
https://github.com/nginx/nginx.git
synced 2024-12-01 11:19:00 +08:00
separate ngx_http_upstream_test_connect() and
use it in ngx_http_upstream_process_header() path too
This commit is contained in:
parent
2c8f05737a
commit
24a95944a7
@ -21,6 +21,7 @@ static void ngx_http_upstream_send_request(ngx_http_request_t *r,
|
||||
ngx_http_upstream_t *u);
|
||||
static void ngx_http_upstream_send_request_handler(ngx_event_t *wev);
|
||||
static void ngx_http_upstream_process_header(ngx_event_t *rev);
|
||||
static ngx_int_t ngx_http_upstream_test_connect(ngx_connection_t *c);
|
||||
static void ngx_http_upstream_process_body_in_memory(ngx_event_t *rev);
|
||||
static void ngx_http_upstream_send_response(ngx_http_request_t *r,
|
||||
ngx_http_upstream_t *u);
|
||||
@ -757,8 +758,7 @@ ngx_http_upstream_reinit(ngx_http_request_t *r, ngx_http_upstream_t *u)
|
||||
static void
|
||||
ngx_http_upstream_send_request(ngx_http_request_t *r, ngx_http_upstream_t *u)
|
||||
{
|
||||
int rc, err;
|
||||
socklen_t len;
|
||||
ngx_int_t rc;
|
||||
ngx_connection_t *c;
|
||||
|
||||
c = u->peer.connection;
|
||||
@ -766,41 +766,9 @@ ngx_http_upstream_send_request(ngx_http_request_t *r, ngx_http_upstream_t *u)
|
||||
ngx_log_debug0(NGX_LOG_DEBUG_HTTP, c->log, 0,
|
||||
"http upstream send request");
|
||||
|
||||
if (!u->request_sent) {
|
||||
|
||||
#if (NGX_HAVE_KQUEUE)
|
||||
|
||||
if (ngx_event_flags & NGX_USE_KQUEUE_EVENT) {
|
||||
if (c->write->pending_eof) {
|
||||
(void) ngx_connection_error(c, c->write->kq_errno,
|
||||
"kevent() reported that connect() failed");
|
||||
ngx_http_upstream_next(r, u, NGX_HTTP_UPSTREAM_FT_ERROR);
|
||||
return;
|
||||
}
|
||||
|
||||
} else
|
||||
#endif
|
||||
{
|
||||
err = 0;
|
||||
len = sizeof(int);
|
||||
|
||||
/*
|
||||
* BSDs and Linux return 0 and set a pending error in err
|
||||
* Solaris returns -1 and sets errno
|
||||
*/
|
||||
|
||||
if (getsockopt(c->fd, SOL_SOCKET, SO_ERROR, (void *) &err, &len)
|
||||
== -1)
|
||||
{
|
||||
err = ngx_errno;
|
||||
}
|
||||
|
||||
if (err) {
|
||||
(void) ngx_connection_error(c, err, "connect() failed");
|
||||
ngx_http_upstream_next(r, u, NGX_HTTP_UPSTREAM_FT_ERROR);
|
||||
return;
|
||||
}
|
||||
}
|
||||
if (!u->request_sent && ngx_http_upstream_test_connect(c) != NGX_OK) {
|
||||
ngx_http_upstream_next(r, u, NGX_HTTP_UPSTREAM_FT_ERROR);
|
||||
return;
|
||||
}
|
||||
|
||||
c->log->action = "sending request to upstream";
|
||||
@ -945,6 +913,11 @@ ngx_http_upstream_process_header(ngx_event_t *rev)
|
||||
return;
|
||||
}
|
||||
|
||||
if (!u->request_sent && ngx_http_upstream_test_connect(c) != NGX_OK) {
|
||||
ngx_http_upstream_next(r, u, NGX_HTTP_UPSTREAM_FT_ERROR);
|
||||
return;
|
||||
}
|
||||
|
||||
if (u->buffer.start == NULL) {
|
||||
u->buffer.start = ngx_palloc(r->pool, u->conf->buffer_size);
|
||||
if (u->buffer.start == NULL) {
|
||||
@ -1277,6 +1250,48 @@ ngx_http_upstream_process_header(ngx_event_t *rev)
|
||||
}
|
||||
|
||||
|
||||
static ngx_int_t
|
||||
ngx_http_upstream_test_connect(ngx_connection_t *c)
|
||||
{
|
||||
int err;
|
||||
socklen_t len;
|
||||
|
||||
#if (NGX_HAVE_KQUEUE)
|
||||
|
||||
if (ngx_event_flags & NGX_USE_KQUEUE_EVENT) {
|
||||
if (c->write->pending_eof) {
|
||||
(void) ngx_connection_error(c, c->write->kq_errno,
|
||||
"kevent() reported that connect() failed");
|
||||
return NGX_ERROR;
|
||||
}
|
||||
|
||||
} else
|
||||
#endif
|
||||
{
|
||||
err = 0;
|
||||
len = sizeof(int);
|
||||
|
||||
/*
|
||||
* BSDs and Linux return 0 and set a pending error in err
|
||||
* Solaris returns -1 and sets errno
|
||||
*/
|
||||
|
||||
if (getsockopt(c->fd, SOL_SOCKET, SO_ERROR, (void *) &err, &len)
|
||||
== -1)
|
||||
{
|
||||
err = ngx_errno;
|
||||
}
|
||||
|
||||
if (err) {
|
||||
(void) ngx_connection_error(c, err, "connect() failed");
|
||||
return NGX_ERROR;
|
||||
}
|
||||
}
|
||||
|
||||
return NGX_OK;
|
||||
}
|
||||
|
||||
|
||||
static void
|
||||
ngx_http_upstream_process_body_in_memory(ngx_event_t *rev)
|
||||
{
|
||||
|
Loading…
Reference in New Issue
Block a user