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

QUIC support #34

Merged
merged 4 commits into from
Nov 29, 2018
Merged

QUIC support #34

merged 4 commits into from
Nov 29, 2018

Conversation

vyzo
Copy link
Collaborator

@vyzo vyzo commented Nov 29, 2018

Adds support for the QUIC transport, enabled with the -quic flag.

Closes #33.

@ghost ghost assigned vyzo Nov 29, 2018
@ghost ghost added the in progress label Nov 29, 2018
@vyzo vyzo requested review from bigs and raulk November 29, 2018 15:16
@vyzo vyzo mentioned this pull request Nov 29, 2018
@raulk
Copy link
Member

raulk commented Nov 29, 2018

@vyzo I assume you tested the packaging. There was an issue reported here: libp2p/go-libp2p-quic-transport#33

@raulk
Copy link
Member

raulk commented Nov 29, 2018

I guess it doesn't matter that much because we expect people to build the daemon with make.

@vyzo
Copy link
Collaborator Author

vyzo commented Nov 29, 2018

yeah, we build after gx-go rw so that issue doesn't affect us.

@vyzo
Copy link
Collaborator Author

vyzo commented Nov 29, 2018

ok, this not working yet.

@raulk
Copy link
Member

raulk commented Nov 29, 2018

Yeah, jumped the gun too soon. Looks like a generalised test failure.

@vyzo
Copy link
Collaborator Author

vyzo commented Nov 29, 2018

updated the gx dependencies, but now when I start the daemon with -quic the other transports disappear.

@vyzo
Copy link
Collaborator Author

vyzo commented Nov 29, 2018

ok, it seems that I have to explicitly specify listen addresses when I explicitly specify transports.
That's a bug in the libp2p constructor I would say.

@vyzo
Copy link
Collaborator Author

vyzo commented Nov 29, 2018

alright, now it should be good. perhaps we should have a quicOnly mode that disables the other listen addrs?

@vyzo vyzo merged commit dd60de8 into master Nov 29, 2018
@ghost ghost removed the in progress label Nov 29, 2018
@vyzo vyzo deleted the feat/quic branch November 29, 2018 15:58
@raulk raulk mentioned this pull request Nov 29, 2018
@raulk
Copy link
Member

raulk commented Nov 29, 2018

@vyzo I started responding here, but I realised we better track the options discussion in a separate issue: #35

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.

None yet

2 participants