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.

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

Use relay node with integrated signalling and connection improvements #303

Merged
merged 6 commits into from
Feb 2, 2023
Merged

Use relay node with integrated signalling and connection improvements #303

merged 6 commits into from
Feb 2, 2023

Conversation

nikugogoi
Copy link
Collaborator

@nikugogoi nikugogoi commented Jan 25, 2023 •
edited by prathamesh0
Loading

Part of #289

  • Requires changes in js-libp2p to successfully close connections
  • Ensure connection with all remote peers on network interrupt
    • Hangup peer connections if heartbeat check fails
    • Browser peers reconnect on discovery
  • Fix multiple connections between two peers
    • Prioritize direct webrtc connection between peers
  • Use cerc-io/webrtc-direct protocol having peer signalling integrated into the relay node

@nikugogoi nikugogoi self-assigned this Jan 25, 2023
@nikugogoi nikugogoi changed the title [WIP] Ensure connection with all remote peers on network interrupt [WIP] Fix connection issues between peers Jan 27, 2023
@nikugogoi nikugogoi changed the title [WIP] Fix connection issues between peers [WIP] Improvements in connection between peers Feb 1, 2023
@nikugogoi nikugogoi mentioned this pull request Feb 2, 2023
Merged
nikugogoi and others added 5 commits February 2, 2023 18:23
@nikugogoi @prathamesh0
Add connection check for all remote peers
06fec0c
@nikugogoi @prathamesh0
Fix for multiple connections between two peers (#305)
6b3a992 
* Close existing connection

* Fix closing existing connections

* Close new connections from only peer

* Refresh in intervals to update UI stats
@nikugogoi @prathamesh0
Prioritize direct webrtc connection between peers (#306)
42e2934 
* Prioritize direct webrtc connection between peers

* Close event listener and add comment
@nikugogoi @prathamesh0
Use scoped libp2p package after publishing to gitea (#309)
6ee078b
@prathamesh0
Use webrtc-direct protocol with peer signalling integrated into relay…
fd2ba93 
… node (#307)

* Use webrtc-direct transport with pubsub based discovery

* Use changes to integrate signalling using relay node

* Add an indicator for peer connection type (direct/relayed)

* Catch errors while creting a libp2p node

* Catch errors while dialling peers

* Catch errors when dialling from relay node

* Pass CLI arg for port to listen on for relay node

* Subscribe chat CLI to pubsub topic

* Update yarn lockfile

* Update webrtc-direct dependency in package json

* Update webrtc-direct version

* Update yarn lockfile
@prathamesh0 prathamesh0 changed the title [WIP] Improvements in connection between peers Improvements in connection between peers Feb 2, 2023
@prathamesh0 prathamesh0 changed the title Improvements in connection between peers Use relay node with integrated signalling and connection improvements Feb 2, 2023
@prathamesh0 prathamesh0 changed the title Use relay node with integrated signalling and connection improvements [WIP] Use relay node with integrated signalling and connection improvements Feb 2, 2023
@prathamesh0 prathamesh0 mentioned this pull request Feb 2, 2023
Merged
@prathamesh0 prathamesh0 changed the title [WIP] Use relay node with integrated signalling and connection improvements Use relay node with integrated signalling and connection improvements Feb 2, 2023
@ashwinphatak ashwinphatak merged commit f5ff7cc into main Feb 2, 2023
@ashwinphatak ashwinphatak deleted the ng-conn-improv branch February 2, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikugogoi nikugogoi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nikugogoi @ashwinphatak @prathamesh0