feat(cli): default the base URL to the REST service address

Use the same default REST address in the CLI when config omits `node.rest_service_address` and when `--base-url` is not provided.

Author: vincenzopalazzo

ldk-server-cli/src/config.rs

@@ -14,6 +14,7 @@ use serde::{Deserialize, Serialize};
 const DEFAULT_CONFIG_FILE: &str = "config.toml";
 const DEFAULT_CERT_FILE: &str = "tls.crt";
 const API_KEY_FILE: &str = "api_key";
+pub const DEFAULT_REST_SERVICE_ADDRESS: &str = "127.0.0.1:3536";
 
 pub fn get_default_data_dir() -> Option<PathBuf> {
 	#[cfg(target_os = "macos")]
@@ -66,6 +67,7 @@ pub struct TlsConfig {
 
 #[derive(Debug, Deserialize)]
 pub struct NodeConfig {
+	#[serde(default = "default_rest_service_address")]
 	pub rest_service_address: String,
 	network: String,
 }
@@ -99,3 +101,53 @@ pub fn load_config(path: &PathBuf) -> Result<Config, String> {
 	toml::from_str(&contents)
 		.map_err(|e| format!("Failed to parse config file '{}': {}", path.display(), e))
 }
+
+pub fn resolve_base_url(cli_base_url: Option<String>, config: Option<&Config>) -> String {
+	cli_base_url
+		.or_else(|| config.map(|config| config.node.rest_service_address.clone()))
+		.unwrap_or_else(default_rest_service_address)
+}
+
+fn default_rest_service_address() -> String {
+	DEFAULT_REST_SERVICE_ADDRESS.to_string()
+}
+
+#[cfg(test)]
+mod tests {
+	use super::{resolve_base_url, Config, DEFAULT_REST_SERVICE_ADDRESS};
+
+	#[test]
+	fn config_defaults_rest_service_address() {
+		let config: Config = toml::from_str(
+			r#"
+				[node]
+				network = "regtest"
+			"#,
+		)
+		.unwrap();
+
+		assert_eq!(config.node.rest_service_address, DEFAULT_REST_SERVICE_ADDRESS);
+	}
+
+	#[test]
+	fn resolve_base_url_uses_cli_arg_first() {
+		let config: Config = toml::from_str(
+			r#"
+				[node]
+				network = "regtest"
+				rest_service_address = "127.0.0.1:3002"
+			"#,
+		)
+		.unwrap();
+
+		assert_eq!(
+			resolve_base_url(Some("127.0.0.1:4000".to_string()), Some(&config)),
+			"127.0.0.1:4000"
+		);
+	}
+
+	#[test]
+	fn resolve_base_url_falls_back_to_default() {
+		assert_eq!(resolve_base_url(None, None), DEFAULT_REST_SERVICE_ADDRESS);
+	}
+}

ldk-server-cli/src/main.rs

@@ -13,7 +13,8 @@ use clap::{CommandFactory, Parser, Subcommand};
 use clap_complete::{generate, Shell};
 use config::{
 	api_key_path_for_storage_dir, cert_path_for_storage_dir, get_default_api_key_path,
-	get_default_cert_path, get_default_config_path, load_config,
+	get_default_cert_path, get_default_config_path, load_config, resolve_base_url,
+	DEFAULT_REST_SERVICE_ADDRESS,
 };
 use hex_conservative::DisplayHex;
 use ldk_server_client::client::LdkServerClient;
@@ -79,7 +80,13 @@ const DEFAULT_DIR: &str = if cfg!(target_os = "macos") {
 	override_usage = "ldk-server-cli [OPTIONS] <COMMAND>"
 )]
 struct Cli {
-	#[arg(short, long, help = "Base URL of the server. If not provided, reads from config file")]
+	#[arg(
+		short,
+		long,
+		help = format!(
+			"Base URL of the server. Defaults to config file or {DEFAULT_REST_SERVICE_ADDRESS}"
+		)
+	)]
 	base_url: Option<String>,
 
 	#[arg(short, long, help = format!("API key for authentication. Defaults by reading {DEFAULT_DIR}/[network]/api_key"))]
@@ -570,12 +577,7 @@ async fn main() {
 		});
 
 	// Get base URL from argument then from config file
-	let base_url =
-		cli.base_url.or_else(|| config.as_ref().map(|c| c.node.rest_service_address.clone()))
-			.unwrap_or_else(|| {
-				eprintln!("Base URL not provided. Use --base-url or ensure config file exists at {DEFAULT_DIR}/config.toml");
-				std::process::exit(1);
-			});
+	let base_url = resolve_base_url(cli.base_url, config.as_ref());
 
 	// Get TLS cert path from argument, then from config tls.cert_path, then from storage dir,
 	// then try default location.