comparison src/http/v3/ngx_http_v3_tables.c @ 8766:70701edfc5d4 quic

HTTP/3: fixed decoder stream stubs. 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.
author Roman Arutyunyan <arut@nginx.com>
date Tue, 04 May 2021 13:38:59 +0300
parents 279ad36f2f4b
children 33ec97749b5f
comparison
equal deleted inserted replaced
8765:687445803d0f 8766:70701edfc5d4
388 ngx_http_v3_ack_header(ngx_connection_t *c, ngx_uint_t stream_id) 388 ngx_http_v3_ack_header(ngx_connection_t *c, ngx_uint_t stream_id)
389 { 389 {
390 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0, 390 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
391 "http3 ack header %ui", stream_id); 391 "http3 ack header %ui", stream_id);
392 392
393 /* XXX */ 393 /* we do not use dynamic tables */
394 394
395 return NGX_OK; 395 return NGX_HTTP_V3_ERR_DECODER_STREAM_ERROR;
396 } 396 }
397 397
398 398
399 ngx_int_t 399 ngx_int_t
400 ngx_http_v3_cancel_stream(ngx_connection_t *c, ngx_uint_t stream_id) 400 ngx_http_v3_cancel_stream(ngx_connection_t *c, ngx_uint_t stream_id)
401 { 401 {
402 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0, 402 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
403 "http3 cancel stream %ui", stream_id); 403 "http3 cancel stream %ui", stream_id);
404 404
405 /* XXX */ 405 /* we do not use dynamic tables */
406 406
407 return NGX_OK; 407 return NGX_OK;
408 } 408 }
409 409
410 410
412 ngx_http_v3_inc_insert_count(ngx_connection_t *c, ngx_uint_t inc) 412 ngx_http_v3_inc_insert_count(ngx_connection_t *c, ngx_uint_t inc)
413 { 413 {
414 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0, 414 ngx_log_debug1(NGX_LOG_DEBUG_HTTP, c->log, 0,
415 "http3 increment insert count %ui", inc); 415 "http3 increment insert count %ui", inc);
416 416
417 /* XXX */ 417 /* we do not use dynamic tables */
418 418
419 return NGX_OK; 419 return NGX_HTTP_V3_ERR_DECODER_STREAM_ERROR;
420 } 420 }
421 421
422 422
423 ngx_int_t 423 ngx_int_t
424 ngx_http_v3_lookup_static(ngx_connection_t *c, ngx_uint_t index, 424 ngx_http_v3_lookup_static(ngx_connection_t *c, ngx_uint_t index,