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

The HTTP/3 protocol on iroh #17

Merged
merged 6 commits into from
Dec 10, 2024
Merged

The HTTP/3 protocol on iroh #17

merged 6 commits into from
Dec 10, 2024

Conversation

flub
Copy link
Contributor

@flub flub commented Oct 31, 2024

This implements a low-level API from hyper which can be used directly. It also includes primitive support for running an axum server.

The latter does some extra copies and puts too much in memory. It should still be generalised into more generic hyper support with parametrised body etc.

This currently relies on a git reference to iroh-net. The 0.28 iroh-net release should fix this, probably best to wait until that release to merge this so that this does not break CI for the other examples randomly as there's no Cargo.lock checked in.

flub added 5 commits October 24, 2024 18:17
There's no Cargo.lock file though so this is painful.  Next release
should solve this.
@matheus23
Copy link
Member

Should we add the h3::quic trait impls to iroh-net under a feature flag?

@flub
Copy link
Contributor Author

flub commented Oct 31, 2024

Should we add the h3::quic trait impls to iroh-net under a feature flag?

oh, i hadn't considered that. I think it might be too early. I don't think those traits are 1.0 yet and putting them in an unstable feature flag is a bit meh and sets too much precedence for throwing things into iroh-net. Should be fine as a separate crate for now.

But if we want to take this more serious we should indeed give it its own repo and some docs etc.

@dignifiedquire
Copy link
Contributor

@flub do we want to merge this?

@flub flub merged commit 77222ae into main Dec 10, 2024
1 check passed
@flub
Copy link
Contributor Author

flub commented Dec 10, 2024

@flub do we want to merge this?

totally!

@dignifiedquire dignifiedquire deleted the flub/h3-iroh branch December 11, 2024 10:49
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.

3 participants