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

Port to http 1.0 and tower-http 0.5 #78

Merged
merged 4 commits into from
Dec 7, 2023
Merged

Conversation

davidpdrsn
Copy link
Contributor

Not sure how to run it but at least it compiles.

@davidpdrsn
Copy link
Contributor Author

Ah here is a graceful shutdown example https://github.com/hyperium/hyper/blob/master/examples/graceful_shutdown.rs. I'll incorporate that here.

@jplatte
Copy link
Owner

jplatte commented Nov 21, 2023

Thanks, it seems to work fine even without graceful shutdown! I believe I added that because Ctrl+C used to not work without it (when browser tab still open, due to keepalive or whatever).

The way to run it is just cargo xtask serve.

Copy link
Owner

@jplatte jplatte left a comment

Choose a reason for hiding this comment

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

I'll wait for the release and the graceful shutdown thing (seems nice to have) before merging but clearly things work and it's not an awful lot of boilerplate :)

Not exactly needed for this use case.
Ctrl+C cancels reliably, which is all that matters.
@jplatte jplatte merged commit 4b4f6de into jplatte:main Dec 7, 2023
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