Skip to content

p2p: fix forwarding block of proxied validator issue in EVN;#3101

Merged
zzzckck merged 11 commits intobnb-chain:developfrom
galaio:new-broadcast
May 19, 2025
Merged

p2p: fix forwarding block of proxied validator issue in EVN;#3101
zzzckck merged 11 commits intobnb-chain:developfrom
galaio:new-broadcast

Conversation

@galaio
Copy link
Contributor

@galaio galaio commented May 16, 2025

Description

This PR will fix forwarding block of proxied validator issue in EVN.

Previously, the validator need to register its NodeID and Sentry NodeID on chain, then the sentry can forwarding the validator's mined block normally.

This PR adds block receiving info for later source checking.

Example

[Node.P2P]
ProxyedValidatorAddresses = ["0x..."]

Changes

Notable changes:

  • chore: support proxyed validator generation;
  • p2p: fix proxyed broadcast issue in EVN;
  • p2p: fix block receive info missing issue;

@galaio galaio requested review from buddh0 and zzzckck May 16, 2025 11:14
@buddh0
Copy link
Contributor

buddh0 commented May 16, 2025

Now, it only checks the block source NodeID with the ProxyedValidator in the config.

still check the sentry nodeid is registered on chain now from code

buddh0
buddh0 previously approved these changes May 16, 2025
@buddh0
Copy link
Contributor

buddh0 commented May 19, 2025

ProxyedValidatorNodeIDs

change desc ProxyedValidatorNodeIDs

buddh0
buddh0 previously approved these changes May 19, 2025
zzzckck
zzzckck previously approved these changes May 19, 2025
@galaio galaio dismissed stale reviews from zzzckck and buddh0 via 19a045b May 19, 2025 06:20
@zzzckck zzzckck merged commit 0cc64d6 into bnb-chain:develop May 19, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants