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

Upgrade to hyper v1 #1088

Open
urkle opened this issue Jan 20, 2024 · 6 comments
Open

Upgrade to hyper v1 #1088

urkle opened this issue Jan 20, 2024 · 6 comments
Labels
feature New feature or request

Comments

@urkle
Copy link
Contributor

urkle commented Jan 20, 2024

Now that hyper 1.0 is out we should upgrade warp to use the new version of hyper.

I have attempted locally to perform this update and there are a number of items that need to be reworked due to the removal of many high-level items from hyper.

How do you want to approach a PR upgrading Hyper? e.g. I can push up what I have so far and we can work through the compile errors.

@urkle urkle added the feature New feature or request label Jan 20, 2024
@urkle
Copy link
Contributor Author

urkle commented Jan 31, 2024

@seanmonstar I have an in-progress PR #1090 .

There are a few issues though.

One big one is figuring out what the Response type should be. e.g. should it be a Response? (the new name for the old Body struct). or should it be some type of BxBody?

The issue I had with BxBodyempty was even with that I couldn't coerce a http_body_util::Empty has a different error class (Infallible) over the error type for the Incoming strict, thus using http_body_util::Empty in place of Incoming::empty() does not work with that.

Then there is the whole gutting of the server and stream stuff out of Hyper that will need to be rewritten/replaced.

@kimhanbeom
Copy link

@urkle @seanmonstar
Does warp currently have any plans for hyper 1.x?

@seanmonstar
Copy link
Owner

I make occasional progress as time allows, though many my users and client direct my attention elsewhere.

@urkle
Copy link
Contributor Author

urkle commented Apr 25, 2024

@seanmonstar if you can find some time to look at my PR and provide suggestions/feedback on the items I posted there that would be helpful so I can move forward on that PR

alexrudy added a commit to alexrudy/tonic that referenced this issue May 26, 2024
Warp is much less popular than Axum today - 30,000 downloads/day on crates.io vs Axum's 200,000/day, as a rough quantiative measure.

As well, warp does not yet have compatiblity with hyper-1.0, though there is some work ongoing (seanmonstar/warp#1088).

When warp adds compatibility with hyper 1.0, these examples can be restored.
github-merge-queue bot pushed a commit to hyperium/tonic that referenced this issue May 26, 2024
Warp is much less popular than Axum today - 30,000 downloads/day on crates.io vs Axum's 200,000/day, as a rough quantiative measure.

As well, warp does not yet have compatiblity with hyper-1.0, though there is some work ongoing (seanmonstar/warp#1088).

When warp adds compatibility with hyper 1.0, these examples can be restored.
@czy-29
Copy link

czy-29 commented May 30, 2024

Upgrading hyper to v1 can also close this issue: #1100

@kushudai
Copy link

Hi @seanmonstar, sorry to keep badgering you, I know you maintain a ton of stuff (I cannot imagine how tbh, it seems insane how many things you're involved in) but if you could spare some time to take a look, that would unblock a lot of us who are trying to upgrade http and hyper.

I, personally, am currently running into StatusCode comparison errors because the types are different because warp is still using http 0.2 😅

We're happy to help but would appreciate some guidance on how to move forward :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants