QUIC: idle mode for main connection.
Now main QUIC connection for HTTP/3 always has c->idle flag set. This allows the connection to receive worker shutdown notification. It is passed to application level via a new conf->shutdown() callback. The HTTP/3 shutdown callback sends GOAWAY to client and gracefully shuts down the QUIC connection.
This commit is contained in:
parent
7cccd64670
commit
595a642018
5 changed files with 57 additions and 4 deletions
|
@ -341,6 +341,7 @@ ngx_quic_new_connection(ngx_connection_t *c, ngx_quic_conf_t *conf,
|
|||
return NULL;
|
||||
}
|
||||
|
||||
c->idle = 1;
|
||||
ngx_reusable_connection(c, 1);
|
||||
|
||||
ngx_log_debug0(NGX_LOG_DEBUG_EVENT, c->log, 0,
|
||||
|
@ -420,9 +421,19 @@ ngx_quic_input_handler(ngx_event_t *rev)
|
|||
}
|
||||
|
||||
if (c->close) {
|
||||
qc->error = NGX_QUIC_ERR_NO_ERROR;
|
||||
qc->error_reason = "graceful shutdown";
|
||||
ngx_quic_close_connection(c, NGX_ERROR);
|
||||
c->close = 0;
|
||||
|
||||
if (!ngx_exiting) {
|
||||
qc->error = NGX_QUIC_ERR_NO_ERROR;
|
||||
qc->error_reason = "graceful shutdown";
|
||||
ngx_quic_close_connection(c, NGX_ERROR);
|
||||
return;
|
||||
}
|
||||
|
||||
if (!qc->closing && qc->conf->shutdown) {
|
||||
qc->conf->shutdown(c);
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
|
@ -28,6 +28,9 @@
|
|||
#define NGX_QUIC_STREAM_UNIDIRECTIONAL 0x02
|
||||
|
||||
|
||||
typedef void (*ngx_quic_shutdown_pt)(ngx_connection_t *c);
|
||||
|
||||
|
||||
typedef enum {
|
||||
NGX_QUIC_STREAM_SEND_READY = 0,
|
||||
NGX_QUIC_STREAM_SEND_SEND,
|
||||
|
@ -74,6 +77,8 @@ typedef struct {
|
|||
ngx_int_t stream_reject_code_uni;
|
||||
ngx_int_t stream_reject_code_bidi;
|
||||
|
||||
ngx_quic_shutdown_pt shutdown;
|
||||
|
||||
u_char av_token_key[NGX_QUIC_AV_KEY_LEN];
|
||||
u_char sr_token_key[NGX_QUIC_SR_KEY_LEN];
|
||||
} ngx_quic_conf_t;
|
||||
|
|
|
@ -141,6 +141,7 @@ struct ngx_http_v3_session_s {
|
|||
uint64_t next_push_id;
|
||||
uint64_t max_push_id;
|
||||
uint64_t goaway_push_id;
|
||||
uint64_t next_request_id;
|
||||
|
||||
off_t total_bytes;
|
||||
off_t payload_bytes;
|
||||
|
@ -158,6 +159,7 @@ void ngx_http_v3_init(ngx_connection_t *c);
|
|||
void ngx_http_v3_reset_connection(ngx_connection_t *c);
|
||||
ngx_int_t ngx_http_v3_init_session(ngx_connection_t *c);
|
||||
ngx_int_t ngx_http_v3_check_flood(ngx_connection_t *c);
|
||||
void ngx_http_v3_shutdown(ngx_connection_t *c);
|
||||
|
||||
ngx_int_t ngx_http_v3_read_request_body(ngx_http_request_t *r);
|
||||
ngx_int_t ngx_http_v3_read_unbuffered_request_body(ngx_http_request_t *r);
|
||||
|
|
|
@ -249,6 +249,8 @@ ngx_http_v3_create_srv_conf(ngx_conf_t *cf)
|
|||
h3scf->quic.stream_reject_code_bidi = NGX_HTTP_V3_ERR_REQUEST_REJECTED;
|
||||
h3scf->quic.active_connection_id_limit = NGX_CONF_UNSET_UINT;
|
||||
|
||||
h3scf->quic.shutdown = ngx_http_v3_shutdown;
|
||||
|
||||
return h3scf;
|
||||
}
|
||||
|
||||
|
|
|
@ -100,6 +100,37 @@ ngx_http_v3_init(ngx_connection_t *c)
|
|||
}
|
||||
|
||||
|
||||
void
|
||||
ngx_http_v3_shutdown(ngx_connection_t *c)
|
||||
{
|
||||
ngx_http_v3_session_t *h3c;
|
||||
|
||||
ngx_log_debug0(NGX_LOG_DEBUG_HTTP, c->log, 0, "http3 shutdown");
|
||||
|
||||
h3c = ngx_http_v3_get_session(c);
|
||||
|
||||
if (h3c == NULL) {
|
||||
ngx_quic_finalize_connection(c, NGX_HTTP_V3_ERR_NO_ERROR,
|
||||
"connection shutdown");
|
||||
return;
|
||||
}
|
||||
|
||||
if (!h3c->goaway) {
|
||||
h3c->goaway = 1;
|
||||
|
||||
#if (NGX_HTTP_V3_HQ)
|
||||
if (!h3c->hq)
|
||||
#endif
|
||||
{
|
||||
(void) ngx_http_v3_send_goaway(c, h3c->next_request_id);
|
||||
}
|
||||
|
||||
ngx_http_v3_shutdown_connection(c, NGX_HTTP_V3_ERR_NO_ERROR,
|
||||
"connection shutdown");
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
static void
|
||||
ngx_http_v3_init_request_stream(ngx_connection_t *c)
|
||||
{
|
||||
|
@ -140,6 +171,8 @@ ngx_http_v3_init_request_stream(ngx_connection_t *c)
|
|||
|
||||
pc = c->quic->parent;
|
||||
|
||||
h3c->next_request_id = c->quic->id + 0x04;
|
||||
|
||||
if (n + 1 == clcf->keepalive_requests
|
||||
|| ngx_current_msec - pc->start_time > clcf->keepalive_time)
|
||||
{
|
||||
|
@ -149,7 +182,7 @@ ngx_http_v3_init_request_stream(ngx_connection_t *c)
|
|||
if (!h3c->hq)
|
||||
#endif
|
||||
{
|
||||
if (ngx_http_v3_send_goaway(c, (n + 1) << 2) != NGX_OK) {
|
||||
if (ngx_http_v3_send_goaway(c, h3c->next_request_id) != NGX_OK) {
|
||||
ngx_http_close_connection(c);
|
||||
return;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue