-
-
Notifications
You must be signed in to change notification settings - Fork 723
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
Comments
@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. |
@urkle @seanmonstar |
I make occasional progress as time allows, though many my users and client direct my attention elsewhere. |
@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 |
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.
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.
Upgrading hyper to v1 can also close this issue: #1100 |
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 :) |
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.
The text was updated successfully, but these errors were encountered: