-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
move go-libp2p-mplex here #1450
Conversation
add support for the new Reset method.
remove conn.Serve
feat: update stream interfaces
change OpenStream to accept a context
update go-mplex, use the context passed to OpenStream
ed6a9b5
to
d6a12cb
Compare
@@ -15,6 +15,8 @@ import ( | |||
"testing" | |||
"time" | |||
|
|||
mplex "github.com/libp2p/go-libp2p/p2p/muxer/mplex" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need the rename?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package is named peerstream_multiplex
, so I think we do.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, can we, like, fix that. Now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. We should probably do the same for yamux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably.
ac02b5c
to
9034260
Compare
9034260
to
663cf70
Compare
Part of #1187.