feat: Add a lower block bound to check_provider_factory_health

flashbots / rbuilder

rbuilder is a blazingly fast, state of the art Ethereum MEV-Boost block builder written in Rust

Apache License 2.0

186 stars 21 forks source link

feat: Add a lower block bound to check_provider_factory_health #32

Closed ferranbt closed 2 weeks ago

ferranbt commented 2 weeks ago

📝 Summary

If the current block number is less than 256, the check_provider_factory_health will overflow and check for the block of the max uint which will not exists. This can happen in devnets with a few blocks.

✅ I have completed the following steps:

[x] Run make lint
[x] Run make test
[x] Added tests (if applicable)

github-actions[bot] commented 2 weeks ago

Benchmark results for `9c514d7`

Report: https://flashbots-rbuilder-ci-stats.s3.us-east-2.amazonaws.com/benchmark/9c514d7-a25f33b/report/index.html


Date (UTC)	2024-07-05T07:58:16+00:00
Commit	`9c514d75acd124ac316fa18d3f42df47e08ff2b7`
Base SHA	`a25f33b44d80a9ea12ac5ea2e68e54a89ce092a8`

Significant changes

None