Skip to content

chore(deps)!: bump h2 from 0.5.10 to 0.6.2#4460

Merged
cratelyn merged 1 commit into
mainfrom
kate/update-h2-crate
Mar 24, 2026
Merged

chore(deps)!: bump h2 from 0.5.10 to 0.6.2#4460
cratelyn merged 1 commit into
mainfrom
kate/update-h2-crate

Conversation

@cratelyn
Copy link
Copy Markdown
Member

@cratelyn cratelyn commented Mar 23, 2026

this crate updates the h2 crate.

while this is mechanically a routine dependency upgrade, this contains
some noteworthy bug fixes that are worth calling out. see
linkerd/linkerd2#14800 for additional
context.

see hyperium/h2#858 and hyperium/h2#860.

We don't want to remember the streams for forever, that would cause
memory to grow. Nor do we want to accept frames on those streams
without limit, since that would waste resources. Thus, we limit how
much we remember, by stream count and duration.

This PR changes the defaults to be something perhaps a little more
reasonable:

  • 50 streams, a conservative amount of half of the recommending
    minimum concurrent streams any remote SHOULD support.

  • 1 second, since the RFC only suggests about 1 RTT. We don't keep
    track of RTT, but even p99 RTT is around 250ms, maybe 500ms.

because this PR changes some defaults in h2, this is marked as a
"breaking" change.


@cratelyn cratelyn self-assigned this Mar 23, 2026
this crate updates the `h2` crate.

while this is mechanically a routine dependency upgrade, this contains
some noteworthy bug fixes that are worth calling out. see
linkerd/linkerd2#14800 for additional
context.

see hyperium/h2#858 and hyperium/h2#860.

> We don't want to remember the streams for forever, that would cause
> memory to grow. Nor do we want to accept frames on those streams
> without limit, since that would waste resources. Thus, we limit how
> much we remember, by stream count and duration.
>
> This PR changes the defaults to be something perhaps a little more
> reasonable:
>
> * 50 streams, a conservative amount of half of the recommending
>   minimum concurrent streams any remote SHOULD support.
>
> * 1 second, since the RFC only suggests about 1 RTT. We don't keep
>   track of RTT, but even p99 RTT is around 250ms, maybe 500ms.

because this PR changes some defaults in `h2`, this is marked as a
"breaking" change.

this fixes linkerd/linkerd2#14908.

X-ref: linkerd/linkerd2#14908
X-ref: hyperium/h2#858
X-ref: hyperium/h2#860
X-ref: hyperium/h2#853
X-ref: linkerd/linkerd2#14800
Signed-off-by: katelyn martin <kate@buoyant.io>
@cratelyn cratelyn force-pushed the kate/update-h2-crate branch from 2db8bc9 to 1f81c77 Compare March 23, 2026 19:43
@cratelyn cratelyn marked this pull request as ready for review March 23, 2026 20:24
@cratelyn cratelyn requested a review from a team as a code owner March 23, 2026 20:24
@cratelyn cratelyn removed the request for review from zaharidichev March 24, 2026 15:14
@cratelyn cratelyn merged commit 1ec95c7 into main Mar 24, 2026
15 checks passed
@cratelyn cratelyn deleted the kate/update-h2-crate branch March 24, 2026 16:41
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