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

[4.0] P2P: fix head_num reporting #1345

Merged
merged 1 commit into from
Jun 28, 2023
Merged

[4.0] P2P: fix head_num reporting #1345

merged 1 commit into from
Jun 28, 2023

Conversation

heifner
Copy link
Member

@heifner heifner commented Jun 27, 2023

P2P notice_message.known_trx catch_up does not report head_num, so do not update head_num reporting. none message does. Regardless only update head_num if a valid value.

Resolves #1328

@heifner heifner linked an issue Jun 27, 2023 that may be closed by this pull request
4.0.2: cleos net peers shows incorrect head_num #1328
Closed
@heifner heifner added the OCI Work exclusive to OCI team label Jun 27, 2023
@heifner heifner merged commit ef910f7 into release/4.0 Jun 28, 2023
16 checks passed
@heifner heifner deleted the GH-1328-p2p-4.0 branch June 28, 2023 11:06
@heifner heifner mentioned this pull request Jun 28, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dimas1185 dimas1185 dimas1185 approved these changes

@linh2931 linh2931 linh2931 approved these changes

Assignees
No one assigned
Labels
OCI Work exclusive to OCI team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4.0.2: cleos net peers shows incorrect head_num
3 participants
@heifner @dimas1185 @linh2931