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

Failed validating block body - Invalid ommers hash #1752

Closed
kdeme opened this issue Sep 15, 2023 · 3 comments
Closed

Failed validating block body - Invalid ommers hash #1752

kdeme opened this issue Sep 15, 2023 · 3 comments
Labels
bug Something isn't working Fluffy

Comments

@kdeme
Copy link
Contributor

kdeme commented Sep 15, 2023

Our testnet fleet has quite a spam of this warning:

WRN 2023-09-12 11:22:13.860+00:00 Failed validating block body               topics="portal_hist" error="Invalid ommers hash"

These are incoming block bodies through offer/accept/utp flow. They are probably coming from a Trin bridge + Trin node.

However, recently added portal-hive tests hit this issue also for post cappella blocks. So it is easily reproducible in portal-hive by running portal-interop simulator:

./hive --sim portal-interop --client fluffy

The strange thing there is that the issue only occurs on gossip tests and not on raw offer testing.

@mynameisdaniil mynameisdaniil self-assigned this Sep 15, 2023
@kdeme
Copy link
Contributor Author

kdeme commented Sep 15, 2023

First thing to investigate here should probably be checking the decoded post Shanghai block its uncles field.
We do a check here if it is empty or not: https://github.com/status-im/nimbus-eth1/blob/master/fluffy/network/history/history_network.nim#L246

@kdeme
Copy link
Contributor Author

kdeme commented Sep 16, 2023

I just realised that the uncles field in the PortalBlockBodyShanghai is a ByteList holding the rlp encoded uncles.
That len() check there assumed checking the actually array I believe, hence this is faulty, but easy to fix.

Still strange that portal-hive offers work but gossip fails. Something seems wrong there too.

We also should have this tested better locally.

@kdeme kdeme added bug Something isn't working Fluffy labels Sep 19, 2023
@kdeme
Copy link
Contributor Author

kdeme commented Sep 19, 2023

Fixed in #1756

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fluffy
Projects
None yet
Development

No branches or pull requests

2 participants