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

peer: remove maybeSendNodeAnn #7525

Conversation

yyforyongyu
Copy link
Member

Upon reconnection, an old node ann is sent which will be ignored by the peer. This PR removes the sending as it's handled by initial sync and re-sync.

@yyforyongyu yyforyongyu added p2p Code related to the peer-to-peer behaviour gossip no-changelog labels Mar 20, 2023
@yyforyongyu yyforyongyu self-assigned this Mar 20, 2023
The node ann is managed by initial sync and the old node announcement
will always be ignored by the peer as it's outdated.
@yyforyongyu yyforyongyu force-pushed the remove-maybe-sent-node-ann branch from 1eae01e to 804100f Compare August 30, 2023 06:48
@yyforyongyu yyforyongyu changed the base branch from 0-16-1-staging to master August 30, 2023 06:48
@yyforyongyu yyforyongyu deleted the remove-maybe-sent-node-ann branch August 31, 2023 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gossip no-changelog p2p Code related to the peer-to-peer behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant