bobanetwork / boba_base

MIT License
1 stars 5 forks source link

Add `/eth/syncing/l2` in dtl #35

Closed boyuan-chen closed 2 years ago

boyuan-chen commented 2 years ago

/eth/syncing returns the syncing status, but the design for this allows the replica to sync up before dtl loads all l2 data. Therefore, the syncing always returns false for l2geth.

This PR introduces the new api endpoint /eth/syncing/l2. This returns the actual syncing status, so we can query the actual highest l2 block that needs to be synced and current transaction index that dtl has synced.

InoMurko commented 2 years ago

Great stuff! Does this go into boba develop too?

boyuan-chen commented 2 years ago

Great stuff! Does this go into boba develop too?

Yep!