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

Make libp2p-utp an happy transport and part of the libp2p blessed family #80

Merged
merged 2 commits into from
Jun 24, 2019

Conversation

daviddias
Copy link
Member

Making some progress on getting libp2p to behave as libp2p needs it to.

@dirkmc
Copy link

dirkmc commented May 23, 2019

@daviddias I just want to make sure you're aware that interface-transport has recently been updated to use async/await and interface-connection is currently being modified. The corresponding WIP js-libp2p-tcp PR is here: libp2p/js-libp2p-tcp#102

@daviddias
Copy link
Member Author

Thanks for letting me know, @dirkmc :)

@alanshaw
Copy link
Member

There's a few TODOs in the code i.e. IPv6 and 0.0.0.0 support. Are they blocking this from being merged?

@daviddias
Copy link
Member Author

@alanshaw it is an improvement on what was there and it doesn't break anything from what existed (in fact, just makes it work). I would be fine merging this now and then finishing the rest of the TODO separately.

@alanshaw alanshaw requested a review from vasco-santos June 24, 2019 11:37
Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

LGTM

@daviddias daviddias merged commit 4e9f068 into master Jun 24, 2019
@daviddias daviddias deleted the happy-transport branch June 24, 2019 15:03
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