FuelLabs/fuel-core

Verify prev block root in the executor and not only in the importer

Open

#2.961 geöffnet am 21. Apr. 2025

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Rust (2.861 Forks)batch import
good first issuetech-debt

Repository-Metriken

Stars
 (57.084 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8h 53m) (1 gemergte PR in 30 T)

Beschreibung

https://github.com/FuelLabs/fuel-core/blob/926b50bcb70ecf3f5ba6acc984a01d4be16bb1c4/crates/services/consensus_module/src/block_verifier.rs#L52-L73

defines the verify_block_fields function that is ran whenever a block is imported by a validator/node. We want to include this verification in the executor as well.

Contributor Guide

Research-Richtung
Finde die Funktion verify block fields in crates/services/consensus module/src/block verifier.rs. Verstehe, wie sie derzeit während des Blockimports aufgerufen wird. Finde dann den Blockverarbeitungspfad des Executors und füge dort einen Aufruf von verify block fields hinzu, um sicherzustellen, dass sie vor oder während der Ausführung läuft. Prüfe auf vorhandene Tests und erwäge, einen Test für die Verifizierung im Executor hinzuzufügen. Siehe die in der Issue verlinkte Datei für die aktuelle Implementierung.
Tech Stack
rust
Domain
backendblockchain
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust programmingBlock verification concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60