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

net: updated check to connect nodes #183

Merged
merged 1 commit into from
Jan 23, 2024
Merged

net: updated check to connect nodes #183

merged 1 commit into from
Jan 23, 2024

Conversation

edtubbs
Copy link
Collaborator

@edtubbs edtubbs commented Jan 11, 2024

Update to auto-reconnect on Error connecting to node, maintaining threshold.

DEBUG: Statecheck: amount of connected nodes: 4
DEBUG: received command from node 11: inv
DEBUG: received command from node 16: ping
DEBUG: sending message to node 16: pong
DEBUG: received command from node 16: inv
DEBUG: received command from node 25: inv
DEBUG: Get inv request with 14 items
DEBUG: Event callback on node 12
DEBUG: Error connecting to node 12.
DEBUG: Connected nodes: 3
DEBUG: Statecheck: amount of connected nodes: 3

@edtubbs edtubbs force-pushed the 0.1.3-dev-node-connect branch 4 times, most recently from bdd7f61 to 8be43b3 Compare January 11, 2024 19:51
@edtubbs
Copy link
Collaborator Author

edtubbs commented Jan 11, 2024

Auto-reconnect on disconnect..

DEBUG: Event callback on node 9
DEBUG: Disconnected from the remote peer 9.
DEBUG: Trying to connect to 8...
DEBUG: Trying to connect to 9...
DEBUG: Trying to connect to 12...
DEBUG: Connected nodes: 21
DEBUG: Event callback on node 9
DEBUG: Successful connected to node 9.
DEBUG: sending message to node 9: version
DEBUG: Connected nodes: 22
DEBUG: Event callback on node 8
DEBUG: Successful connected to node 8.
DEBUG: sending message to node 8: version
DEBUG: Connected nodes: 23

@edtubbs
Copy link
Collaborator Author

edtubbs commented Jan 12, 2024

Another successful auto-reconnect on x86_64...

DEBUG: Statecheck: amount of connected nodes: 8
DEBUG: Event callback on node 1
DEBUG: Disconnected from the remote peer 1.
DEBUG: Trying to connect to 1...
DEBUG: Trying to connect to 2...
DEBUG: Trying to connect to 5...
DEBUG: Connected nodes: 7
DEBUG: Event callback on node 5
DEBUG: Successful connected to node 5.
DEBUG: sending message to node 5: version
DEBUG: Connected nodes: 8
DEBUG: Event callback on node 2
DEBUG: Successful connected to node 2.
DEBUG: sending message to node 2: version
DEBUG: Connected nodes: 9

@edtubbs
net: updated check to connect nodes
1b2b575 
net: updated comment for check
net: removed error state from timeout
spv: removed disconnects from statecheck
michilumin
michilumin approved these changes Jan 23, 2024
View reviewed changes
Copy link
Contributor

@michilumin michilumin left a comment

Choose a reason for hiding this comment

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

simplification and lgtm in re being more straightforward

@michilumin michilumin merged commit 8f22153 into dogecoinfoundation:0.1.3-dev Jan 23, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michilumin michilumin michilumin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@edtubbs @michilumin