Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #10805 zero dynamic table #11445

Merged
merged 5 commits into from
Feb 26, 2024
Merged

Conversation

gregw
Copy link
Contributor

@gregw gregw commented Feb 23, 2024

Fix #10805 zero dynamic table:

  • Set the correct default table size in the context
  • just in case, always send the table size on the first encode

Set the correct default size for the table.
Always send the max table size on the first encode
@gregw
Copy link
Contributor Author

gregw commented Feb 26, 2024

@sbordet @lachlan-roberts nudge

@gregw gregw merged commit 41e53fa into jetty-9.4.x Feb 26, 2024
7 checks passed
@gregw gregw deleted the fix/jetty-9.4.x/10805/ZeroDynamicTable branch February 26, 2024 18:09
gregw added a commit that referenced this pull request Feb 27, 2024
* Added test for #10805 Zero Dynamic Table

* fixed file header

* Added test for #10805 Zero Dynamic Table

* Fix for #10805 Zero Dynamic Table

Set the correct default size for the table.
Always send the max table size on the first encode
gregw added a commit that referenced this pull request Feb 28, 2024
* Fix #10805 zero dynamic table (#11445)

* Added test for #10805 Zero Dynamic Table

* fixed file header

* Added test for #10805 Zero Dynamic Table

* Fix for #10805 Zero Dynamic Table

Set the correct default size for the table.
Always send the max table size on the first encode

* updated file header

Signed-off-by: gregw <gregw@webtide.com>

---------

Signed-off-by: gregw <gregw@webtide.com>
gregw added a commit that referenced this pull request Feb 28, 2024
* Fix #10805 zero dynamic table (#11445)

* Added test for #10805 Zero Dynamic Table

* fixed file header

* Added test for #10805 Zero Dynamic Table

* Fix for #10805 Zero Dynamic Table

Set the correct default size for the table.
Always send the max table size on the first encode

* updated file header

Signed-off-by: gregw <gregw@webtide.com>

---------

Signed-off-by: gregw <gregw@webtide.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Jetty response with an invalid HTTP2 packet if the client set the hpack table size as 0
2 participants