Skip to content

Commit

Permalink
Add patch for ZERO signatures in legacy DB
Browse files Browse the repository at this point in the history
  • Loading branch information
clabby committed Nov 19, 2023
1 parent 37f8f8d commit e3e14c8
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 7 deletions.
8 changes: 4 additions & 4 deletions Cargo.lock

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

8 changes: 6 additions & 2 deletions crates/db-utils/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,14 @@ authors.workspace = true
version.workspace = true

[dependencies]
leveldb = { git = "https://github.com/cassc/leveldb", branch = "allow-impl-database-key" }
alloy-primitives = "0.4.2"
alloy-rlp = "0.3.3"
reth-primitives = { git = "https://github.com/paradigmxyz/reth" }

# Contains a patch that allows for implementing the `Key` trait, which we need for Geth's variable-length'd DB keys.
leveldb = { git = "https://github.com/cassc/leveldb", branch = "allow-impl-database-key" }

# Contains a patch that allows for 0 signatures, which exist in the legacy DB
reth-primitives = { git = "https://github.com/clabby/reth", branch = "cl/op-legacy-signature" }

# Misc
anyhow = "1.0.70"
Expand Down
1 change: 0 additions & 1 deletion crates/db-utils/src/leveldb/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,6 @@ mod db_test {
let database: Database<DBKey> = Database::open(db_path.as_path(), options).unwrap();
let reader = GethDBReader::new(database);

// TODO: We need to make a patch for legacy deposits, there's no signature in those.
for i in BEDROCK_TRANSITION - FULL_PRUNE_DEPTH - 1..=BEDROCK_TRANSITION {
if let Err(e) = reader.block_by_number(i) {
panic!("Error reading block # {}: {}", i, e);
Expand Down

0 comments on commit e3e14c8

Please sign in to comment.