Apply Clippy auto-fixes via cargo clippy --fix

This change was automatically applied using `cargo clippy --fix && cargo fmt`.
It addresses the newly introduced `uninlined_format_args`
lint, which requires variables to be inlined directly within format
strings as of Rust 1.88.0 nightly.

[Clippy lint
reference](https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args)

Author: arminsabouri

payjoin-cli/src/app/config.rs

@@ -71,7 +71,7 @@ impl Config {
                         "Multiple version flags specified. Please use only one of: {}",
                         Self::VERSION_FLAGS
                             .iter()
-                            .map(|(flag, _)| format!("--{}", flag))
+                            .map(|(flag, _)| format!("--{flag}"))
                             .collect::<Vec<_>>()
                             .join(", ")
                     )));
@@ -163,8 +163,7 @@ impl Config {
                         Ok(v2) => config.version = Some(VersionConfig::V2(v2)),
                         Err(e) =>
                             return Err(ConfigError::Message(format!(
-                                "Valid V2 configuration is required for BIP77 mode: {}",
-                                e
+                                "Valid V2 configuration is required for BIP77 mode: {e}"
                             ))),
                     }
                 }
@@ -182,7 +181,7 @@ impl Config {
             ));
         }
 
-        log::debug!("App config: {:?}", config);
+        log::debug!("App config: {config:?}");
         Ok(config)
     }
 
@@ -315,10 +314,10 @@ where
     match path_str {
         None => Ok(None),
         Some(path) => std::fs::read(path)
-            .map_err(|e| serde::de::Error::custom(format!("Failed to read ohttp_keys file: {}", e)))
+            .map_err(|e| serde::de::Error::custom(format!("Failed to read ohttp_keys file: {e}")))
             .and_then(|bytes| {
                 payjoin::OhttpKeys::decode(&bytes).map_err(|e| {
-                    serde::de::Error::custom(format!("Failed to decode ohttp keys: {}", e))
+                    serde::de::Error::custom(format!("Failed to decode ohttp keys: {e}"))
                 })
             })
             .map(Some),

payjoin-cli/src/app/mod.rs

@@ -43,14 +43,14 @@ pub trait App: Send + Sync {
     }
 
     fn process_pj_response(&self, psbt: Psbt) -> Result<bitcoin::Txid> {
-        log::debug!("Proposed psbt: {:#?}", psbt);
+        log::debug!("Proposed psbt: {psbt:#?}");
 
         let signed = self.wallet().process_psbt(&psbt)?;
         let tx = self.wallet().finalize_psbt(&signed)?;
 
         let txid = self.wallet().broadcast_tx(&tx)?;
 
-        println!("Payjoin sent. TXID: {}", txid);
+        println!("Payjoin sent. TXID: {txid}");
         Ok(txid)
     }
 }
@@ -83,7 +83,7 @@ fn read_local_cert() -> Result<Vec<u8>> {
 
 async fn handle_interrupt(tx: watch::Sender<()>) {
     if let Err(e) = signal::ctrl_c().await {
-        eprintln!("Error setting up Ctrl-C handler: {}", e);
+        eprintln!("Error setting up Ctrl-C handler: {e}");
     }
     let _ = tx.send(());
 }

payjoin-cli/src/app/v2.rs

@@ -152,7 +152,7 @@ impl App {
         let mut pj_uri = session.pj_uri();
         pj_uri.amount = amount;
         println!("Request Payjoin by sharing this Payjoin Uri:");
-        println!("{}", pj_uri);
+        println!("{pj_uri}");
 
         let mut interrupt = self.interrupt.clone();
         let receiver = tokio::select! {
@@ -208,8 +208,8 @@ impl App {
                             println!("No response yet.");
                         }
                         Err(re) => {
-                            println!("{}", re);
-                            log::debug!("{:?}", re);
+                            println!("{re}");
+                            log::debug!("{re:?}");
                             return Err(anyhow!("Response error").context(re));
                         }
                     }
@@ -223,8 +223,8 @@ impl App {
                 match v1_ctx.process_response(&mut response.bytes().await?.to_vec().as_slice()) {
                     Ok(psbt) => Ok(psbt),
                     Err(re) => {
-                        println!("{}", re);
-                        log::debug!("{:?}", re);
+                        println!("{re}");
+                        log::debug!("{re:?}");
                         Err(anyhow!("Response error").context(re))
                     }
                 }
@@ -286,7 +286,7 @@ impl App {
             self.config.max_fee_rate,
         )?;
         let payjoin_proposal_psbt = payjoin_proposal.psbt();
-        log::debug!("Receiver's Payjoin proposal PSBT Rsponse: {:#?}", payjoin_proposal_psbt);
+        log::debug!("Receiver's Payjoin proposal PSBT Rsponse: {payjoin_proposal_psbt:#?}");
         Ok(payjoin_proposal)
     }
 }

payjoin-cli/src/db/error.rs

@@ -20,13 +20,13 @@ pub(crate) enum Error {
 impl fmt::Display for Error {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         match self {
-            Error::Sled(e) => write!(f, "Database operation failed: {}", e),
+            Error::Sled(e) => write!(f, "Database operation failed: {e}"),
             #[cfg(feature = "v2")]
-            Error::Serialize(e) => write!(f, "Serialization failed: {}", e),
+            Error::Serialize(e) => write!(f, "Serialization failed: {e}"),
             #[cfg(feature = "v2")]
-            Error::Deserialize(e) => write!(f, "Deserialization failed: {}", e),
+            Error::Deserialize(e) => write!(f, "Deserialization failed: {e}"),
             #[cfg(feature = "v2")]
-            Error::NotFound(key) => write!(f, "Key not found: {}", key),
+            Error::NotFound(key) => write!(f, "Key not found: {key}"),
         }
     }
 }

payjoin-directory/src/db.rs

@@ -26,8 +26,8 @@ impl std::fmt::Display for Error {
         use Error::*;
 
         match &self {
-            Redis(error) => write!(f, "Redis error: {}", error),
-            Timeout(timeout) => write!(f, "Timeout: {}", timeout),
+            Redis(error) => write!(f, "Redis error: {error}"),
+            Timeout(timeout) => write!(f, "Timeout: {timeout}"),
         }
     }
 }
@@ -49,7 +49,7 @@ pub(crate) type Result<T> = core::result::Result<T, Error>;
 
 impl DbPool {
     pub async fn new(timeout: Duration, db_host: String) -> Result<Self> {
-        let client = Client::open(format!("redis://{}", db_host))?;
+        let client = Client::open(format!("redis://{db_host}"))?;
         Ok(Self { client, timeout })
     }
 

payjoin-directory/src/key_config.rs

@@ -74,7 +74,7 @@ pub fn read_server_config(dir: &Path) -> Result<ServerKeyConfig> {
 /// Get the path to the key configuration file
 /// For now, default to [KEY_ID].ikm.
 /// In the future this might be able to save multiple keys named by KeyId.
-fn key_path(dir: &Path) -> PathBuf { dir.join(format!("{}.ikm", KEY_ID)) }
+fn key_path(dir: &Path) -> PathBuf { dir.join(format!("{KEY_ID}.ikm")) }
 
 #[cfg(test)]
 mod tests {

payjoin-directory/src/lib.rs

@@ -142,7 +142,7 @@ pub async fn listen_tcp_with_tls(
     cert_key: (Vec<u8>, Vec<u8>),
     ohttp: ohttp::Server,
 ) -> Result<tokio::task::JoinHandle<Result<(), BoxError>>, BoxError> {
-    let addr = format!("0.0.0.0:{}", port);
+    let addr = format!("0.0.0.0:{port}");
     let listener = tokio::net::TcpListener::bind(&addr).await?;
     listen_tcp_with_tls_on_listener(listener, db_host, timeout, cert_key, ohttp).await
 }
@@ -352,7 +352,7 @@ async fn post_fallback_v1(
         Err(_) => return Ok(bad_request_body_res),
     };
 
-    let v2_compat_body = format!("{}\n{}", body_str, query);
+    let v2_compat_body = format!("{body_str}\n{query}");
     let id = ShortId::from_str(id)?;
     pool.push_default(&id, v2_compat_body.into())
         .await

payjoin-test-utils/src/lib.rs

@@ -129,7 +129,7 @@ pub async fn init_directory(
     (u16, tokio::task::JoinHandle<std::result::Result<(), BoxSendSyncError>>),
     BoxSendSyncError,
 > {
-    println!("Database running on {}", db_host);
+    println!("Database running on {db_host}");
     let timeout = Duration::from_secs(2);
     let ohttp_server = payjoin_directory::gen_ohttp_server_config()?;
     payjoin_directory::listen_tcp_with_tls_on_free_port(
@@ -221,7 +221,7 @@ pub fn init_bitcoind_multi_sender_single_reciever(
 ) -> Result<(bitcoind::BitcoinD, Vec<bitcoincore_rpc::Client>, bitcoincore_rpc::Client), BoxError> {
     let bitcoind = init_bitcoind()?;
     let wallets_to_create =
-        (0..number_of_senders + 1).map(|i| (format!("sender_{}", i), None)).collect::<Vec<_>>();
+        (0..number_of_senders + 1).map(|i| (format!("sender_{i}"), None)).collect::<Vec<_>>();
     let mut wallets = create_and_fund_wallets(&bitcoind, wallets_to_create)?;
     let receiver = wallets.pop().expect("receiver to exist");
     let senders = wallets;

payjoin/src/hpke.rs

@@ -303,8 +303,7 @@ impl fmt::Display for HpkeError {
             PayloadTooLarge { actual, max } => {
                 write!(
                     f,
-                    "Plaintext too large, max size is {} bytes, actual size is {} bytes",
-                    max, actual
+                    "Plaintext too large, max size is {max} bytes, actual size is {actual} bytes"
                 )
             }
             PayloadTooShort => write!(f, "Payload too small"),

payjoin/src/into_url.rs

@@ -12,7 +12,7 @@ impl std::fmt::Display for Error {
 
         match self {
             BadScheme => write!(f, "URL scheme is not allowed"),
-            ParseError(e) => write!(f, "{}", e),
+            ParseError(e) => write!(f, "{e}"),
         }
     }
 }