Merge tag 'v0.15.2'

Pathfinder v0.15.2

CHANGELOG.md

@@ -7,6 +7,13 @@ More expansive patch notes and explanations may be found in the specific [pathfi
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.15.2] - 2024-12-04
+
+### Fixed
+
+- Pathfinder fails to properly do a reorg due to a SQL statement referring a table that does not exist.
+- `--rpc.get-events-max-uncached-bloom-filters-to-load` setting is ineffective.
+
 ## [0.15.1] - 2024-12-02
 
 ### Fixed

Cargo.toml

@@ -43,7 +43,7 @@ lto = true
 opt-level = 3
 
 [workspace.package]
-version = "0.15.1"
+version = "0.15.2"
 edition = "2021"
 license = "MIT OR Apache-2.0"
 rust-version = "1.80"
@@ -75,7 +75,6 @@ const-decoder = "0.3.0"
 const_format = "0.2.31"
 criterion = "0.5.1"
 dashmap = "6.1"
-env_logger = "0.10.0"
 fake = "2.8.0"
 ff = "0.13"
 flate2 = "1.0.27"
@@ -128,7 +127,7 @@ starknet_api = { git = "https://github.com/cartridge-gg/sequencer", branch = "ca
 starknet-types-core = "=0.1.5"
 syn = "1.0"
 tempfile = "3.8"
-test-log = { version = "0.2.12", default-features = false }
+test-log = { version = "0.2.12", features = ["trace"] }
 thiserror = "1.0.48"
 time = "0.3.36"
 tokio = "1.37.0"

crates/p2p/Cargo.toml

@@ -60,12 +60,11 @@ zeroize = { workspace = true }
 
 [dev-dependencies]
 clap = { workspace = true, features = ["derive", "env", "wrap_help"] }
-env_logger = { workspace = true }
 hex = { workspace = true }
 rand = { workspace = true }
 rstest = { workspace = true }
 tagged = { path = "../tagged" }
 tagged-debug-derive = { path = "../tagged-debug-derive" }
-test-log = { workspace = true, features = ["trace"] }
+test-log = { workspace = true }
 tokio = { workspace = true, features = ["test-util"] }
 tracing-subscriber = { workspace = true, features = ["env-filter"] }

crates/p2p/src/client/peer_agnostic/tests.rs

@@ -112,8 +112,6 @@ async fn make_header_stream(
     #[case] responses: Vec<Result<(TestPeer, Vec<BlockHeadersResponse>), TestPeer>>,
     #[case] expected_stream: Vec<(TestPeer, SignedBlockHeader)>,
 ) {
-    let _ = env_logger::builder().is_test(true).try_init();
-
     for (reverse, direction) in [(false, "forward"), (true, "backward")] {
         let (peers, responses) = unzip_fixtures(responses.clone());
         let get_peers = move || {
@@ -295,7 +293,6 @@ async fn make_transaction_stream(
     #[case] num_txns_per_block: Vec<usize>,
     #[case] expected_stream: Vec<Result<(TestPeer, Vec<TestTxn>), ()>>,
 ) {
-    let _ = env_logger::builder().is_test(true).try_init();
     let (peers, responses) = unzip_fixtures(responses);
     let get_peers = move || {
         let peers = peers.clone();
@@ -518,7 +515,6 @@ async fn make_state_diff_stream(
     #[case] state_diff_len_per_block: Vec<usize>,
     #[case] expected_stream: Vec<Result<(TestPeer, StateUpdateData), ()>>,
 ) {
-    let _ = env_logger::builder().is_test(true).try_init();
     let (peers, responses) = unzip_fixtures(responses);
     let get_peers = move || {
         let peers = peers.clone();
@@ -713,7 +709,6 @@ async fn make_class_definition_stream(
     #[case] declared_classes_per_block: Vec<usize>,
     #[case] expected_stream: Vec<Result<(TestPeer, ClassDefinition), ()>>,
 ) {
-    let _ = env_logger::builder().is_test(true).try_init();
     let (peers, responses) = unzip_fixtures(responses);
     let get_peers = move || {
         let peers = peers.clone();
@@ -904,7 +899,6 @@ async fn make_event_stream(
     #[case] events_per_block: Vec<usize>,
     #[case] expected_stream: Vec<Result<(TestPeer, TaggedEventsForBlockByTransaction), ()>>,
 ) {
-    let _ = env_logger::builder().is_test(true).try_init();
     let (peers, responses) = unzip_fixtures(responses);
     let get_peers = move || {
         let peers = peers.clone();