Commit 1ec95c7
authored
chore(deps)!: bump h2 from 0.5.10 to 0.6.2 (#4460)
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>1 parent 6f672fa commit 1ec95c7
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
838 | 838 | | |
839 | 839 | | |
840 | 840 | | |
841 | | - | |
| 841 | + | |
842 | 842 | | |
843 | | - | |
| 843 | + | |
844 | 844 | | |
845 | 845 | | |
846 | 846 | | |
| |||
1040 | 1040 | | |
1041 | 1041 | | |
1042 | 1042 | | |
1043 | | - | |
| 1043 | + | |
1044 | 1044 | | |
1045 | 1045 | | |
1046 | 1046 | | |
| |||
0 commit comments