Skip to content
This repository has been archived by the owner on Jul 1, 2023. It is now read-only.

Add quic as a transport #32

Open
liamsi opened this issue Jul 23, 2021 · 2 comments
Open

Add quic as a transport #32

liamsi opened this issue Jul 23, 2021 · 2 comments

Comments

@liamsi
Copy link
Contributor

liamsi commented Jul 23, 2021

IMO, it would be cool to add QUIC as a transport too already. I briefly skimmed through the go implementation: should be straight-forward (could also just be a boolean flag for now; same for the rust impl).

Orthogonal to that but while at it, the (go) dependencies should also be updated maybe.
In case you want, I have a branch with updated go-deps here: https://github.com/liamsi/libp2p-perf/tree/update-deps

@mxinden
Copy link
Owner

mxinden commented Jul 23, 2021

👋 Hi Ismail,

Cool to see this used by others as well.

IMO, it would be cool to add QUIC as a transport too already. I briefly skimmed through the go implementation: should be straight-forward (could also just be a boolean flag for now; same for the rust impl).

Indeed. Rust's libp2p implementation does not yet support QUIC, though it is in progress. Do you want to create a patch for the Golang implementation?

Orthogonal to that but while at it, the (go) dependencies should also be updated maybe. In case you want, I have a branch with updated go-deps here: https://github.com/liamsi/libp2p-perf/tree/update-deps

With #33 @dependabot should now take care of it. If not, I will cherry-pick your patch, thanks.

@mxinden
Copy link
Owner

mxinden commented Oct 19, 2021

Crossreferencing #51 here which will be adding QUIC support to libp2p-perf.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants