EVM: Set FinalizedHash correctly in `forkChoiceUpdatedV3` call in ExecuteTxs
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- go
- Domaine
- blockchain
Piste de recherche
Commencez dans l’implémentation EVM de ExecuteTxs, au premier appel à forkChoiceUpdatedV3, puis lisez PR 2394 ainsi que les chemins de démarrage de Rollkit Store et engineClient mentionnés dans l’issue. Déterminez si l’état finalisé doit provenir de DA Included Height ou de hashes persistés, et définissez les vérifications de cohérence au démarrage pour les hashes de head, safe et finalized. Le travail est terminé lorsque les redémarrages et les indexeurs reçoivent des hashes de blocs cohérents.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In https://github.com/rollkit/rollkit/pull/2394, in order to have sequencer restarts work well, we modified the first call to forkChoiceUpdatedV3 in the ExecuteTxs EVM implementation to assign the previous block hash to HeadBlockHash, SafeBlockHash, and FinalizedBlockHash.
We should revisit this and take one or more of the following approaches:
- Set the FinalizedBlockHash based on the DA Included Height from the Rollkit Store. For the Head and Safe block hashes, use the Rollkit height which should be the same as the reth block height but it might be good to actually sanity check that on startup and set them in engineClient correctly on start.
- Persist the current versions of head, safe, and finalizedBlockHashes somewhere so on restarts they can be loaded back easily.
The concrete consequences of setting finalizedHash incorrectly are unknown currently but it'll likely be at least an issue in indexers and block explorers showing inconsistent finalized block hashes for a small period of time.
- Langage dominant
- Go
- Étoiles
- 361
- Forks
- 283
- Merge moyen
- 3 j 5 h
- PR mergées (30 j)
- 14
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de evstack/ev-node
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
Les mainteneurs répondent en général sous 2 jours
-
C:p2p
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
Les mainteneurs répondent en général sous 2 jours
-
good first issue T:enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
evstack/ev-node#1672 · 6 commentaires ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de evstack/ev-node
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 90/100
FootprintAI/Containarium#2338 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: core doesn't build standalone on dev since a17068054 (go-mp3 require dropped, go.sum pruned)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
SagerNet/sing-openvpn#11 ·
-
priority: P3 type: devops
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
jiegui2025/hwspec#57 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
columnar-tech/dbc#513 ·
Les mainteneurs répondent en général sous 1 jour