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.

Sign up for GitHub

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

Jump to bottom

Removal of OpenStream swarm notification #10

Closed

dennis-tra opened this issue May 24, 2022 · 1 comment

Assignees

Labels

Collaborator

dennis-tra commented May 24, 2022 •

edited

Loading

Pull request libp2p/go-libp2p-core#250 removed the OpenStream notification. Punchr uses the notification here: https://github.com/dennis-tra/punchr/blob/c4b2fcf93e63ae4a8656864a399638c0c9641372/cmd/client/host.go#L372

Guess, we'll stick with an older libp2p version then.

dennis-tra added a commit that referenced this issue


          Check opened streams periodically

832cf47

Fixes issue #10

Pull request libp2p/go-libp2p-core#250 removed the OpenStream notification. Punchr uses the notification here: https://github.com/dennis-tra/punchr/blob/c4b2fcf93e63ae4a8656864a399638c0c9641372/cmd/client/host.go#L372

dennis-tra self-assigned this

dennis-tra added the go-client label

dennis-tra added a commit that referenced this issue


          Check opened streams periodically

c9ba867

Fixes issue #10

Pull request libp2p/go-libp2p-core#250 removed the OpenStream notification. Punchr uses the notification here: https://github.com/dennis-tra/punchr/blob/c4b2fcf93e63ae4a8656864a399638c0c9641372/cmd/client/host.go#L372

Collaborator Author

dennis-tra commented Jun 14, 2022

Updated libp2p and added periodic check

dennis-tra closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment