Add the date when the node was started to the name of the logfile.

Create a symlink with the old log name to the current log file.
This makes it easier to delete old logs

Author: ulrichard

src/lib.rs

@@ -161,6 +161,8 @@ use std::convert::TryInto;
 use std::default::Default;
 use std::fs;
 use std::net::ToSocketAddrs;
+use std::os::unix::fs::symlink;
+use std::path::Path;
 use std::sync::atomic::{AtomicBool, Ordering};
 use std::sync::{Arc, Mutex, RwLock};
 use std::time::{Duration, Instant, SystemTime};
@@ -384,8 +386,19 @@ impl Builder {
 		fs::create_dir_all(bdk_data_dir.clone()).expect("Failed to create BDK data directory");
 
 		// Initialize the Logger
-		let log_file_path = format!("{}/ldk_node.log", config.storage_dir_path);
-		let logger = Arc::new(FilesystemLogger::new(log_file_path, config.log_level));
+		let log_file_path = format!(
+			"{}/logs/ldk_node_{}.log",
+			config.storage_dir_path,
+			chrono::offset::Local::now().format("%Y_%m_%d")
+		);
+		let logger = Arc::new(FilesystemLogger::new(log_file_path.clone(), config.log_level));
+		let log_file_symlink = format!("{}/ldk_node.log", config.storage_dir_path);
+		if Path::new(&log_file_symlink).exists() {
+			fs::remove_file(&log_file_symlink)
+				.expect("Failed to remove an old symlink for the log file");
+		}
+		symlink(&log_file_path, &log_file_symlink)
+			.expect(&format!("Failed to create symlink for the log file: {}", log_file_symlink));
 
 		// Initialize the on-chain wallet and chain access
 		let seed_bytes = match &*self.entropy_source_config.read().unwrap() {
@@ -1743,7 +1756,7 @@ impl<K: KVStore + Sync + Send + 'static> Node<K> {
 	/// # use ldk_node::bitcoin::Network;
 	/// # let mut config = Config::default();
 	/// # config.network = Network::Regtest;
-	/// # config.storage_dir_path = "/tmp/ldk_node_test/".to_string();
+	/// # config.storage_dir_path = "/tmp/ldk_node_test_list_payments".to_string();
 	/// # let builder = Builder::from_config(config);
 	/// # let node = builder.build();
 	/// node.list_payments_with_filter(|p| p.direction == PaymentDirection::Outbound);

src/test/functional_tests.rs

@@ -310,6 +310,9 @@ fn start_stop_reinit() {
 	node.sync_wallets().unwrap();
 	assert_eq!(node.onchain_balance().unwrap().get_spendable(), expected_amount.to_sat());
 
+	let log_file_symlink = format!("{}/ldk_node.log", config.storage_dir_path);
+	assert!(std::path::Path::new(&log_file_symlink).is_symlink());
+
 	node.stop().unwrap();
 	assert_eq!(node.stop(), Err(Error::NotRunning));