pinax-network / firehose-beacon

Apache License 2.0
4 stars 0 forks source link

Include missing slots instead of skipping the blocks #23

Open fschoell opened 3 weeks ago

fschoell commented 3 weeks ago

Currently whenever a slot is skipped and we get an error when fetching the block number from lighthouse here, we are skipping that block. However skipped blocks are still having states assigned internally, which are needed in certain cases (for example validating blocks). Instead of just skipping those blocks we should include them as missing slots.

Task list:

suchapalaver commented 3 days ago

Thanks for opening this! Since I reported the issue, we've open-sourced a tool, a Firehose client in Rust, that we've been using in development for using Firehose in Semiotic's verifiable extraction work. We have a simple workaround when using the Beacon chain addressing this issue, so I thought I'd share that here in case it's useful.