ChainSafe / ChainSafe/gossamer

Westend #22648256: invalid signature for precommit in grandpa justification

Open
#4,190 1 comment 0 reactions 0 assignees View on GitHub
C-complex P-critical S-grandpa S-sync-westend T-bug
Dominant language
Go
Stars
454
Forks
144
PR merge metrics
No merged PRs in 30d

Description

## Describe the bug

Gossamer could not successfully validate a Grandpa justification while following tip at block #22648256:

```
2024-09-19T17:57:20Z ERROR processing block #22648256 (0xb0d92d06e6dc1d078ee38ecced208803a869b4e6e31d6402bbb157a225bb31ff) failed: verifying justification: decoding and verifying justification: bad justification for header: invalid signature for precommit in grandpa justification pkg=sync

2024-09-19T17:57:20Z CRITICAL current sync strategy failed with: while handling ready block: verifying justification: decoding and verifying justification: bad justification for header: invalid signature for precommit in grandpa justification pkg=sync

2024-09-19T17:57:20Z INFO [FINISHED] worker 12D3KooWJifoDhCL3swAKt7MWhFb7wLRFD9oG33AL3nAathmU24x, request: BlockRequestMessage RequestedData=19 StartingBlock={22648251} Direction=0 Max=8 pkg=sync

2024-09-19T17:56:54Z INFO 🚣 currently syncing, 49 peers connected, finalized #22641152 (0xc1878bb1...490605cb), best #22648250 (0xc291e489...20beec70) pkg=sync
```

This issue is related and should be fixed once Gossamer starts to receive Commit messages from validators which mean that we will always have the highest finalized block, as we can see the highest finalized block is #22641152 which is not the highest finalized block of the network

Blocked by issue: https://github.com/ChainSafe/gossamer/issues/2931

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.