refactor: replace log directory path to log file path

Author: enigbe

bindings/ldk_node.udl

@@ -5,7 +5,7 @@ namespace ldk_node {
 
 dictionary Config {
 	string storage_dir_path;
-	string? log_dir_path;
+	string? log_file_path;
 	Network network;
 	sequence<SocketAddress>? listening_addresses;
 	NodeAlias? node_alias;

src/builder.rs

@@ -298,9 +298,9 @@ impl NodeBuilder {
 		self
 	}
 
-	/// Sets the log dir path if logs need to live separate from the storage directory path.
-	pub fn set_log_dir_path(&mut self, log_dir_path: String) -> &mut Self {
-		self.config.log_dir_path = Some(log_dir_path);
+	/// Sets the log file path if the log file needs to live separate from the storage directory path.
+	pub fn set_log_file_path(&mut self, log_dir_path: String) -> &mut Self {
+		self.config.log_file_path = Some(log_dir_path);
 		self
 	}
 
@@ -611,8 +611,8 @@ impl ArcedNodeBuilder {
 	}
 
 	/// Sets the log dir path if logs need to live separate from the storage directory path.
-	pub fn set_log_dir_path(&self, log_dir_path: String) {
-		self.inner.write().unwrap().set_log_dir_path(log_dir_path);
+	pub fn set_log_file_path(&self, log_file_path: String) {
+		self.inner.write().unwrap().set_log_file_path(log_file_path);
 	}
 
 	/// Sets the Bitcoin network used.
@@ -1234,13 +1234,13 @@ fn build_with_store_internal(
 /// Sets up the node logger, creating a new log file if it does not exist, or utilizing
 /// the existing log file.
 fn setup_logger(config: &Config) -> Result<Arc<FilesystemLogger>, BuildError> {
-	let log_dir = match &config.log_dir_path {
+	let log_file_path = match &config.log_file_path {
 		Some(log_dir) => String::from(log_dir),
-		None => config.storage_dir_path.clone(),
+		None => format!("{}/{}", config.storage_dir_path.clone(), "ldk_node.log"),
 	};
 
 	Ok(Arc::new(
-		FilesystemLogger::new(log_dir, config.log_level)
+		FilesystemLogger::new(log_file_path, config.log_level)
 			.map_err(|_| BuildError::LoggerSetupFailed)?,
 	))
 }

src/config.rs

@@ -105,8 +105,8 @@ pub struct Config {
 	pub storage_dir_path: String,
 	/// The path where logs are stored.
 	///
-	/// If set to `None`, logs can be found in the `logs` subdirectory in [`Config::storage_dir_path`].
-	pub log_dir_path: Option<String>,
+	/// If set to `None`, logs can be found in [`Config::storage_dir_path`] directory.
+	pub log_file_path: Option<String>,
 	/// The used Bitcoin network.
 	pub network: Network,
 	/// The addresses on which the node will listen for incoming connections.
@@ -167,7 +167,7 @@ impl Default for Config {
 	fn default() -> Self {
 		Self {
 			storage_dir_path: DEFAULT_STORAGE_DIR_PATH.to_string(),
-			log_dir_path: None,
+			log_file_path: None,
 			network: DEFAULT_NETWORK,
 			listening_addresses: None,
 			trusted_peers_0conf: Vec::new(),

src/logger.rs

@@ -22,19 +22,16 @@ pub(crate) struct FilesystemLogger {
 }
 
 impl FilesystemLogger {
-	/// Creates a new filesystem logger given the path to the log file directory and the log level.
-	pub(crate) fn new(log_dir: String, level: Level) -> Result<Self, ()> {
-		let log_file_name = "ldk_node.log";
-		let log_file_path = format!("{}/{}", log_dir, log_file_name);
-
+	/// Creates a new filesystem logger given the path to the log file and the log level.
+	pub(crate) fn new(log_file_path: String, level: Level) -> Result<Self, ()> {
 		if let Some(parent_dir) = Path::new(&log_file_path).parent() {
 			fs::create_dir_all(parent_dir).expect("Failed to create log parent directory");
 
 			// make sure the file exists.
 			fs::OpenOptions::new()
 				.create(true)
 				.append(true)
-				.open(log_file_path.clone())
+				.open(&log_file_path)
 				.map_err(|e| eprintln!("ERROR: Failed to open log file: {}", e))?;
 		}