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

[1.1.2 -> main] P2P: Do not close connection on block nack block_notice #1229

Merged
merged 5 commits into from
Mar 4, 2025

Conversation

heifner
Copy link
Member

@heifner heifner commented Mar 4, 2025

When responding to a block_notice for block nack feature, do not close the connection if the block is not available. Under normal circumstances the block should be available. However, if a fork switch makes the block(s) not available when sending, do not disconnect as this is "expected" during a fork switch.

Merge release/1.1 into main including #1226

Resolves #1224

@heifner heifner requested review from linh2931 and greg7mdp March 4, 2025 19:30
@heifner heifner added the OCI Work exclusive to OCI team label Mar 4, 2025
@heifner heifner merged commit 6258920 into main Mar 4, 2025
36 checks passed
@heifner heifner deleted the GH-1224-p2p-block-nack-close-main branch March 4, 2025 20:27
@ericpassmore
Copy link
Contributor

Note:start
category: Other
component: P2P
summary: Do not close connection on block nack during block_notice.
Note:end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1.1 block nack enqueue block can close connection on fork-switch
4 participants