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

feat: allow enabling http1/http2 individually #80

Merged
merged 3 commits into from
Jan 30, 2024

Conversation

SergioBenitez
Copy link
Contributor

@SergioBenitez SergioBenitez commented Dec 19, 2023

As in the title. Previously we pulled in just the deps for http1 or just the deps for http2. This reenables that.

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.

This could be a way to address hyperium/hyper#3493, if we leave the server-auto feature in. Basically, if you have enabled server and http1, then the auto builder can exist with only the HTTP/1 parts.

What do you think @davidpdrsn?

Cargo.toml Outdated Show resolved Hide resolved
@SergioBenitez
Copy link
Contributor Author

It seems that auto.rs has received some rather major changes. Happy to rebase and fix the fallout as soon as this is generally approved.

@SergioBenitez
Copy link
Contributor Author

Rebased on master.

@ikrivosheev
Copy link

@seanmonstar @SergioBenitez any updates?)

@SergioBenitez
Copy link
Contributor Author

@ikrivosheev There's nothing to do on my end.

@SergioBenitez
Copy link
Contributor Author

@seanmonstar What's preventing this from getting in? I really don't want to keep rebasing as changes are made upstream.

@seanmonstar seanmonstar merged commit 703d8da into hyperium:master Jan 30, 2024
16 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
3 participants