Merge pull request #2049 from mintlayer/update_rust_edition

Update Rust edition to 2024

Author: ImplOfAnImpl

Cargo.lock

@@ -6,7 +6,7 @@ repository = "https://github.com/mintlayer/mintlayer-core"
 readme = "README.md"
 license = "MIT"
 version = "1.3.0"
-edition = "2021"
+edition = "2024"
 
 [workspace]
 members = [
@@ -126,7 +126,7 @@ storage = { path = "storage" }
 utxo = { path = "utxo" }
 
 [workspace.package]
-edition = "2021"
+edition = "2024"
 rust-version = "1.88"
 version = "1.3.0"
 license = "MIT"
@@ -156,7 +156,7 @@ console-subscriber = "0.5"
 criterion = "0.5"
 crossterm = "0.28"
 csv = "1.3"
-ctor = "0.2"
+ctor = "0.10"
 derive_more = { version = "1.0", features = ["full"] }
 directories = "5.0"
 humantime = "2.1"

Cargo.toml

@@ -1038,10 +1038,10 @@ impl ApiServerInMemoryStorage {
 
         // Handle a degenerate case when the block is stored several times using different heights
         // (to be consistent with the postgres implementation).
-        if let Some(previously_stored_height) = previously_stored_height {
-            if previously_stored_height != block_height {
-                self.main_chain_blocks_table.remove(&previously_stored_height);
-            }
+        if let Some(previously_stored_height) = previously_stored_height
+            && previously_stored_height != block_height
+        {
+            self.main_chain_blocks_table.remove(&previously_stored_height);
         }
 
         Ok(())

api-server/api-server-common/src/storage/impls/in_memory/mod.rs

@@ -17,6 +17,7 @@ utils = { path = "../../utils" }
 utils-networking = { path = "../../utils/networking" }
 
 clap = { workspace = true, features = ["derive"] }
+ctor.workspace = true
 thiserror.workspace = true
 tokio = { workspace = true, features = ["full"] }
 

api-server/scanner-daemon/Cargo.toml

@@ -140,14 +140,10 @@ pub enum ApiServerScannerError {
     PostgresConnectionError(ApiServerStorageError),
 }
 
+utils::enable_rust_backtrace!();
+
 #[tokio::main]
 async fn main() -> Result<(), ApiServerScannerError> {
-    utils::rust_backtrace::enable();
-
-    if std::env::var("RUST_LOG").is_err() {
-        std::env::set_var("RUST_LOG", "info");
-    }
-
     let args = ApiServerScannerArgs::parse();
 
     logging::init_logging();

api-server/scanner-daemon/src/main.rs

@@ -19,6 +19,7 @@ serialization = { path = "../../serialization" }
 
 async-trait.workspace = true
 chrono.workspace = true
+ctor.workspace = true
 futures = { workspace = true, default-features = false }
 libtest-mimic.workspace = true
 proptest.workspace = true

api-server/storage-test-suite/Cargo.toml

@@ -20,17 +20,16 @@ use api_server_common::storage::{
     storage_api::ApiServerStorage,
 };
 use common::chain::{config::create_unit_test_config, ChainConfig};
-use utils::rust_backtrace;
 
 #[must_use]
 #[allow(clippy::unused_async)]
 async fn make_in_memory_storage(chain_config: Arc<ChainConfig>) -> impl ApiServerStorage {
     TransactionalApiServerInMemoryStorage::new(&chain_config)
 }
 
-fn main() {
-    rust_backtrace::enable();
+utils::enable_rust_backtrace!();
 
+fn main() {
     let storage_maker = || make_in_memory_storage(Arc::new(create_unit_test_config()));
     let result = api_server_backend_test_suite::run(storage_maker);
 

api-server/storage-test-suite/tests/in_memory.rs

@@ -23,7 +23,6 @@ use api_server_common::storage::{
 };
 use common::chain::{config::create_unit_test_config, ChainConfig};
 use containers::with_container::ApiServerStorageWithContainer;
-use utils::rust_backtrace;
 
 #[must_use]
 async fn make_postgres_storage(chain_config: Arc<ChainConfig>) -> impl ApiServerStorage {
@@ -56,9 +55,9 @@ async fn make_postgres_storage(chain_config: Arc<ChainConfig>) -> impl ApiServer
     ApiServerStorageWithContainer::new(storage, podman)
 }
 
-fn main() {
-    rust_backtrace::enable();
+utils::enable_rust_backtrace!();
 
+fn main() {
     // Only run the test if the env var is defined
     if std::env::var("ML_CONTAINERIZED_TESTS").is_err() {
         eprintln!("Warning: Skipping Postgres containerized tests");

api-server/storage-test-suite/tests/postgres.rs

@@ -20,6 +20,7 @@ mempool = { path = "../../mempool" }
 
 axum.workspace = true
 async-trait.workspace = true
+ctor.workspace = true
 clap = { workspace = true, features = ["derive"] }
 hex.workspace = true
 serde = { workspace = true, features = ["derive"] }

api-server/web-server/Cargo.toml

@@ -36,12 +36,10 @@ use utils::{cookie::COOKIE_FILENAME, default_data_dir::default_data_dir_for_chai
 
 use crate::error::ApiServerWebServerInitError;
 
+utils::enable_rust_backtrace!();
+
 #[tokio::main]
 async fn main() -> Result<(), ApiServerWebServerInitError> {
-    if std::env::var("RUST_LOG").is_err() {
-        std::env::set_var("RUST_LOG", "info");
-    }
-
     logging::init_logging();
 
     let args = ApiServerWebServerConfig::parse();