Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

respect the OpenStream context #30

Closed
marten-seemann opened this issue Dec 18, 2020 · 0 comments · Fixed by #31
Closed

respect the OpenStream context #30

marten-seemann opened this issue Dec 18, 2020 · 0 comments · Fixed by #31
Labels
kind/enhancement A net-new feature or improvement to an existing feature

Comments

@marten-seemann
Copy link
Contributor

#29 adds a context to OpenStream, but doesn't use it.
We should make sure that OpenStream actually returns when the context is canceled. This will probably require changes to go-yamux as well.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
kind/enhancement A net-new feature or improvement to an existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

update go-yamux to v2.0.0, use context passed to OpenStream libp2p/go-libp2p-yamux
1 participant
@marten-seemann