Skip to content

Fix devnet2 finalization issues#74

Merged
ArtiomTr merged 1 commit intograndinetech:mainfrom
ArtiomTr:fix/proposer-signature
Feb 12, 2026
Merged

Fix devnet2 finalization issues#74
ArtiomTr merged 1 commit intograndinetech:mainfrom
ArtiomTr:fix/proposer-signature

Conversation

@ArtiomTr
Copy link
Collaborator

Fixed few finalization issues, encountered in devnet-2:

  • Invalid proposer signature - this was due to invalid signature SSZ representation - previously it was a byte list inside container, now it is just a byte list.
  • Unknown block errors - p2p limits weren't configured properly, and with addition of aggregated signatures (which can be up to 1mb), messages with blocks weren't received.
  • Finalization issues - validator public key was in invalid format, similar to proposer signature error.

Overall this not only fixes all finalization issues related to devnet-2, but also state transition code was refactored a bit to be more similar to spec.

@ArtiomTr ArtiomTr force-pushed the fix/proposer-signature branch from 76dabe0 to ab1427c Compare February 12, 2026 12:52
Fixed few finalization issues, encountered in devnet-2:
* Invalid proposer signature - this was due to invalid signature SSZ
  representation - previously it was a byte list inside container, now
  it is just a byte list.
* Unknown block errors - p2p limits weren't configured properly, and
  with addition of aggregated signatures (which can be up to 1mb),
  messages with blocks weren't received.
* Finalization issues - validator public key was in invalid format,
  similar to proposer signature error.

Overall this not only fixes all finalization issues related to devnet-2,
but also state transition code was refactored a bit to be more similar
to spec.
@ArtiomTr ArtiomTr force-pushed the fix/proposer-signature branch from ab1427c to b3aed1e Compare February 12, 2026 13:09
@ArtiomTr ArtiomTr merged commit 3b4f312 into grandinetech:main Feb 12, 2026
0 of 3 checks passed
@ArtiomTr ArtiomTr deleted the fix/proposer-signature branch February 12, 2026 13:09
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.

1 participant