From af6f6ff85cffa8d931e7e250351ff014eac8cda6 Mon Sep 17 00:00:00 2001 From: teyrebaz33 Date: Fri, 21 Aug 2026 13:42:36 +0300 Subject: [PATCH] chore: remove stale FIXME comments on snapshot URL MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Both FIXMEs read 'Update this to the actual snapshot URL', but the URLs are already the real, working snapshot endpoints — confirmed via crates/snapshots/src/download.rs using the same host without any such disclaimer, and docs/running-an-arc-node.md documenting it directly to users with no caveat. Replaced with a note on the base-prefix/latest.txt object-store pattern per @osr21's review on #275, to pre-empt someone re-adding a FIXME after curling the bare base URL and getting a 404. Fixes #275 --- crates/execution-config/src/defaults.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/execution-config/src/defaults.rs b/crates/execution-config/src/defaults.rs index bbed0d3a..a8945a59 100644 --- a/crates/execution-config/src/defaults.rs +++ b/crates/execution-config/src/defaults.rs @@ -23,7 +23,7 @@ use reth_cli_commands::download::DownloadDefaults; use reth_node_core::args::DefaultRpcServerArgs; use std::borrow::Cow; -// FIXME: Update this to the actual snapshot URL. +// Base prefix for the object store; the resolvable pointer is `/latest.txt`. /// Default snapshot URL for Arc Network testnet (chain ID 5042002). pub(crate) const DEFAULT_DOWNLOAD_URL: &str = "https://snapshots.arc.network/5042002"; @@ -36,7 +36,7 @@ pub const RPC_MAX_SUBSCRIPTIONS_PER_CONNECTION: u32 = 32; fn init_download_urls() { let download_defaults = DownloadDefaults { available_snapshots: vec![ - // FIXME: Update this to the actual snapshot URL. + // Base prefix; the resolvable pointer is `/latest.txt`. Cow::Borrowed("https://snapshots.arc.network/5042002 (testnet)"), Cow::Borrowed("https://snapshots.arc.network/5042001 (devnet)"), ],