CERBERUS/nginx
1
0
Fork 0
mirror of https://github.com/nginx/nginx.git synced 2026-10-23 12:32:56 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

HTTP/3: fixed decoder stream stubs.

Browse source 
Now ngx_http_v3_ack_header() and ngx_http_v3_inc_insert_count() always generate
decoder error.  Our implementation does not use dynamic tables and does not
expect client to send Section Acknowledgement or Insert Count Increment.

Stream Cancellation, on the other hand, is allowed to be sent anyway.  This is
why ngx_http_v3_cancel_stream() does not return an error.
This commit is contained in:
Roman Arutyunyan 2021-05-04 13:38:59 +03:00
parent acc3ad0060
commit bbbc80465b
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/http/v3/ngx_http_v3_tables.c
View file

@ -390,9 +390,9 @@ ngx_http_v3_ack_header(ngx_connection_t *c, ngx_uint_t stream_id)
ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
"http3 ack header %ui", stream_id);
/* XXX */
/* we do not use dynamic tables */
return NGX_OK;
return NGX_HTTP_V3_ERR_DECODER_STREAM_ERROR;
}
@ -402,7 +402,7 @@ ngx_http_v3_cancel_stream(ngx_connection_t *c, ngx_uint_t stream_id)
ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
"http3 cancel stream %ui", stream_id);
/* XXX */
/* we do not use dynamic tables */
return NGX_OK;
}
@ -414,9 +414,9 @@ ngx_http_v3_inc_insert_count(ngx_connection_t *c, ngx_uint_t inc)
ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
"http3 increment insert count %ui", inc);
/* XXX */
/* we do not use dynamic tables */
return NGX_OK;
return NGX_HTTP_V3_ERR_DECODER_STREAM_ERROR;
}