Skip to content

Commit

Permalink
fix: BLOBHASH & BLOBBASEFEE should not enabled in CURIE (#40)
Browse files Browse the repository at this point in the history
* fix blob related issue

* add sepolia test

* fix
  • Loading branch information
lightsing authored Aug 30, 2024
1 parent 6b86fa9 commit b15a5dd
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,10 +83,10 @@ jobs:
uses: actions/checkout@v4
with:
repository: 'scroll-tech/block-testdata'
sparse-checkout: 'mainnet_blocks/flatten-proofs'
path: 'testdata'
- uses: dtolnay/rust-toolchain@master
with:
toolchain: ${{ matrix.rust }}
- uses: Swatinem/rust-cache@v2
- run: cargo run --release --features bin-deps --package stateless-block-verifier --bin stateless-block-verifier -- -k run-file testdata/mainnet_blocks/flatten-proofs/*
- run: cargo run --release --features bin-deps --package stateless-block-verifier --bin stateless-block-verifier -- -k run-file testdata/sepolia_blocks/*
10 changes: 5 additions & 5 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 15 additions & 4 deletions src/utils/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,21 @@ where
{
let mut ok = true;
for account_post_state in exec.account_after.iter() {
let local_acc = db
.basic_ref(account_post_state.address.0.into())
.unwrap()
.unwrap();
let local_acc = match db.basic_ref(account_post_state.address.0.into()).unwrap() {
Some(acc) => acc,
None => {
if account_post_state.balance.is_zero()
&& account_post_state.nonce == 0
&& account_post_state.poseidon_code_hash.is_zero()
&& account_post_state.keccak_code_hash.is_zero()
{
continue;
}
ok = false;
dev_error!("local acc not found, trace acc {account_post_state:?}");
continue;
}
};

#[cfg(feature = "dev")]
if tracing::enabled!(Level::TRACE) {
Expand Down

0 comments on commit b15a5dd

Please sign in to comment.