Skip to content

Fix HPACK table size update sequencing#651

Merged
arturobernalg merged 1 commit intoapache:masterfrom
arturobernalg:hpack-table-size-update-sequencing
Mar 13, 2026
Merged

Fix HPACK table size update sequencing#651
arturobernalg merged 1 commit intoapache:masterfrom
arturobernalg:hpack-table-size-update-sequencing

Conversation

@arturobernalg
Copy link
Member

Signal the minimum pending table size first and the final size second, as required by RFC 7541 section 4.2.

- the smallest maximum table size that occurs in that interval MUST be signaled
- The final maximum size is always signaled

@arturobernalg arturobernalg requested a review from ok2c March 11, 2026 20:01
Copy link
Member

@ok2c ok2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arturobernalg I assume you know what you are doing

Signal the minimum pending table size first and the final size second
@arturobernalg arturobernalg force-pushed the hpack-table-size-update-sequencing branch from 13e318b to 86cbf49 Compare March 13, 2026 12:45
@arturobernalg
Copy link
Member Author

@arturobernalg I assume you know what you are doing

Hi @ok2c I double-checked it against RFC 7541 §4.2 and added regression coverage for repeated table size updates between header blocks.

@arturobernalg arturobernalg merged commit f37ed19 into apache:master Mar 13, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants