trafficserver icon indicating copy to clipboard operation
trafficserver copied to clipboard

Add http2.incomplete_header_timeout_in

Open masaori335 opened this issue 1 year ago • 1 comments

Introduce a new active timeout for incoming HTTP/2 stream. This requires clients to complete sending HTTP headers (send END_HEADERS flag on HEADERS or CONTINUATION frame) within the configured timeout.

Default value is 10s as we talked at ATS Summit in Denver.

Timing of setting and canceling timeouts will be below.

http2timeouts

masaori335 avatar May 15 '24 05:05 masaori335

Looks good, but can we add or alter an Au test to exercise the new code?

ywkaras avatar May 21 '24 16:05 ywkaras

Adding AuTest looks good idea, but let me do that in another PR later. Because we need to have a special client or extend proxy-verifier. ( I have a client to test this code but it's written in go-lang )

masaori335 avatar May 23 '24 06:05 masaori335

Cherry-picked to v10.0.x

cmcfarlen avatar May 24 '24 15:05 cmcfarlen

Do we still need this on 9.2?

ezelkow1 avatar Jan 16 '25 17:01 ezelkow1

@ezelkow1 yes, we still need this for 9.2.x. Let me know if there're complicated conflicts to backport.

masaori335 avatar Jan 17 '25 01:01 masaori335