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

fix(network): subscribing to the Libp2p event bus #828

Merged
merged 6 commits into from Nov 29, 2023
Merged

Conversation

themantre
Copy link
Contributor

Description

Subscribing to the EventBus of LibP2P helped us figure out when we have a healthy direct connection with a peer. This enables us to send the connect event more effectively and initiate handshaking with a peer.

@themantre themantre changed the title Libp2p event bus fix(network): subscribing to the Libp2p event bus Nov 29, 2023
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

Merging #828 (4da4560) into main (19f0063) will decrease coverage by 0.11%.
The diff coverage is 83.87%.

❗ Current head 4da4560 differs from pull request most recent head 56c9bb2. Consider uploading reports for the commit 56c9bb2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #828      +/-   ##
==========================================
- Coverage   82.85%   82.75%   -0.11%     
==========================================
  Files         172      172              
  Lines        8364     8383      +19     
==========================================
+ Hits         6930     6937       +7     
- Misses       1112     1120       +8     
- Partials      322      326       +4     

network/notifee.go Outdated Show resolved Hide resolved
network/notifee.go Outdated Show resolved Hide resolved
network/notifee.go Outdated Show resolved Hide resolved
network/notifee.go Outdated Show resolved Hide resolved
b00f and others added 3 commits November 29, 2023 23:08
Co-authored-by: Rostam <kehiiiiya@gmail.com>
Co-authored-by: Rostam <kehiiiiya@gmail.com>
Co-authored-by: Rostam <kehiiiiya@gmail.com>
Copy link
Contributor

@kehiy kehiy left a comment

Choose a reason for hiding this comment

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

@themantre thanks, everything looks good.

@kehiy kehiy merged commit fc66588 into main Nov 29, 2023
10 checks passed
@kehiy kehiy deleted the libp2p-event-bus branch November 29, 2023 15:24
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.

None yet

3 participants