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

chore(server/conn): backport max_pending_accept_reset_streams() #3796

Open
wants to merge 1 commit into
base: 0.14.x
Choose a base branch
from

Conversation

cratelyn
Copy link

NB: this commit is based on the 0.14.x release series.

the server::conn::http2 submodule provides types that are conditionally compiled when the backports feature is active, to facilitate upgrading to the 1.x release.

i've been working on upgrading the
linkerd2-proxy project to use hyper 1.0, but encountered a pertinent gap in Builder<E>'s interface when setting the backports and deprecated features.

this adds a max_pending_accept_reset_streams(..) method (added in #3201) to this builder, so that 0.14.x users relying on this functionality can prepare to upgrade to hyper 1.x safely.

this is effectively a backport of #3507, which restored this interface to the 1.0 release.

for more information, see:

Copy link
Member

@seanmonstar seanmonstar left a comment

Choose a reason for hiding this comment

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

Thanks!

@seanmonstar
Copy link
Member

Submitted #3797 to fix that MSRV job.

@cratelyn
Copy link
Author

Submitted #3797 to fix that MSRV job.

thank you! i'll rebase now

…builder

NB: this commit is based on the `0.14.x` release series.

the `server::conn::http2` submodule provides types that are
conditionally compiled when the `backports` feature is active, to
facilitate upgrading to the `1.x` release.

i've been working on upgrading the
[`linkerd2-proxy`](https://github.com/linkerd/linkerd2-proxy/) project
to use hyper 1.0, but encountered some gaps in `Builder<E>`'s interface
when setting the `backports` and `deprecated` features.

this adds a `max_pending_accept_reset_streams(..)` method
(_added in #3201_) to this builder, so that 0.14.x users relying on
this functionality can prepare to upgrade to hyper 1.x safely.

this is effectively a backport of hyperium#3507, which restored this interface
to the 1.0 release.

for more information, see:

* hyperium#3201
* hyperium#3507
* hyperium#3461
* https://hyper.rs/guides/1/upgrading/

Signed-off-by: katelyn martin <[email protected]>
@cratelyn cratelyn force-pushed the kate/backport-max-pending-accept-reset-streams branch from 28f5e67 to 403b2d0 Compare November 27, 2024 22:17
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