Skip to content

fix(net): robustness fixes for multiple issues - #6983

Open
xxo1shine wants to merge 1 commit into
tronprotocol:release_v4.8.3from
xxo1shine:fix/net-peer-message-robustness
Open

xxo1shine wants to merge 1 commit into
tronprotocol:release_v4.8.3from
xxo1shine:fix/net-peer-message-robustness

Conversation

@xxo1shine

Copy link
Copy Markdown
Collaborator

What does this PR do?
Validate inbound HELLO messages before logging or handshake processing, and tolerate optional node identity data. Handle concurrent sync queue changes, missing connection failure causes, and accurate transaction diagnostics. Correct the default QPS parameter type so fallback configuration remains usable.

Fixes #6969

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

Validate inbound HELLO messages before logging or handshake processing, and tolerate optional node identity data. Handle concurrent sync queue changes, missing connection failure causes, and accurate transaction diagnostics. Correct the default QPS parameter type so fallback configuration remains usable.
@github-actions
github-actions Bot requested a review from 317787106 September 21, 2026 08:05
@halibobo1205 halibobo1205 added this to the GreatVoyage-v4.8.3 milestone Sep 21, 2026
@halibobo1205 halibobo1205 linked an issue Sep 21, 2026 that may be closed by this pull request
@halibobo1205 halibobo1205 added the topic:net p2p net work, synchronization label Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic:net p2p net work, synchronization

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Robustness fixes for multiple issues

3 participants