Skip to content

feat: add http2_header_table_size and http2_max_concurrent_streams methods#274

Merged
seanmonstar merged 2 commits intohyperium:masterfrom
barjin:master
Mar 2, 2026
Merged

feat: add http2_header_table_size and http2_max_concurrent_streams methods#274
seanmonstar merged 2 commits intohyperium:masterfrom
barjin:master

Conversation

@barjin
Copy link
Contributor

@barjin barjin commented Mar 2, 2026

Follows the pattern set by the other http2_ methods. The project depends on hyper=1.8.0, which has the header_table_size() and max_concurrent_streams() methods (added here). This is omitted for no obvious reason in hyper-utils and reqwest.

Related to apify/impit#385 (will need subsequent PR to reqwest, adding methods for setting these values there).

Copilot AI and others added 2 commits March 2, 2026 16:48
Adds `http2_header_table_size` method to the client `Builder`

---

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: barjin <61918049+barjin@users.noreply.github.com>
Builder in client.rs was missing a method to configure the HTTP/2 SETTINGS_MAX_CONCURRENT_STREAMS value, despite h2_builder exposing max_concurrent_streams.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: barjin <61918049+barjin@users.noreply.github.com>
@seanmonstar seanmonstar merged commit 8ae9e8b into hyperium:master Mar 2, 2026
18 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.

3 participants