EspressoSystems / espresso-sequencer

96 stars 68 forks source link

Fix the getHotShotCommitment #1625

Closed ImJeremyHe closed 3 months ago

ImJeremyHe commented 3 months ago

Before this change: When fetching the snapshot height at the hotshot height that is exactly the previous snapshot height, we will get itself.

This PR:

Get the correct snapshot height for the previous snapshot height

Key places to review:

The logic and the test

How to test this PR

I am not quite sure about this change. But with the espresso-dev-node image built on this branch, things worked well in the arbitrum integration.