Skip to content

Commit

Permalink
fix: allow missing nonce in remote blocks (#539)
Browse files Browse the repository at this point in the history
* fix: allow missing nonce in remote blocks

* misc: add changeset
  • Loading branch information
Wodann authored Jul 3, 2024
1 parent dfdd380 commit b056511
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
5 changes: 5 additions & 0 deletions .changeset/big-boxes-drop.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@nomicfoundation/edr": patch
---

Fixes restriction to disallow remote blocks without a nonce
4 changes: 3 additions & 1 deletion crates/edr_evm/src/block/remote.rs
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,9 @@ impl IntoRemoteBlock<L1ChainSpec> for edr_rpc_eth::Block<edr_rpc_eth::Transactio
// TODO don't accept remote blocks with missing mix hash,
// see https://github.com/NomicFoundation/edr/issues/518
mix_hash: self.mix_hash.unwrap_or_default(),
nonce: self.nonce.ok_or(CreationError::MissingNonce)?,
// TODO don't accept remote blocks with missing nonce,
// see https://github.com/NomicFoundation/edr/issues/518
nonce: self.nonce.unwrap_or_default(),
base_fee_per_gas: self.base_fee_per_gas,
withdrawals_root: self.withdrawals_root,
blob_gas: self.blob_gas_used.and_then(|gas_used| {
Expand Down

0 comments on commit b056511

Please sign in to comment.