FuelLabs/fuel-core

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

Open

#2.961 aberto em 21 de abr. de 2025

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Rust (2.861 forks)batch import
good first issuetech-debt

Métricas do repositório

Stars
 (57.084 stars)
Métricas de merge de PR
 (Mesclagem média 8h 53m) (1 fundiu PR em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Localize a função verify block fields em crates/services/consensus module/src/block verifier.rs. Entenda como ela é atualmente chamada durante a importação de blocos. Em seguida, encontre o caminho de processamento de blocos do executor e adicione uma chamada a verify block fields lá, garantindo que ela seja executada antes ou durante a execução. Verifique se há testes existentes e considere adicionar um teste para a verificação no executor. Consulte o arquivo vinculado na issue para a implementação atual.
Pilha de tecnologia
rust
Domain
backendblockchain
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust programmingBlock verification concepts
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60