fix(spsp_routes): apply formatting suggestions

Signed-off-by: Kristian Wahlroos <kristian.whl@gmail.com>

Author: koivunej

crates/interledger-api/src/routes/accounts.rs

@@ -454,28 +454,28 @@ where
         .and(warp::path::end())
         .and(with_store.clone())
         .and_then(move |id: Uuid, tracking_info: String, store: S| {
-            trace!("Tracking info: {:?}", tracking_info);
 
             let server_secret_clone = server_secret_clone.clone();
             async move {
                 let accounts = store.get_accounts(vec![id]).await?;
                 // TODO return the response without instantiating an SpspResponder (use a simple fn)
 
-                match Address::from_str(&format!("{}.{}", accounts[0].ilp_address(), tracking_info))
+                let combined = format!("{}.{}", accounts[0].ilp_address(), tracking_info);
+                match Address::from_str(&combined)
                 {
                     Ok(addr) => {
                         trace!(
-                            "merged: {}",
-                            format!("{}.{}", accounts[0].ilp_address(), tracking_info)
+                            account_id=%id,
+                            "Appended tracking info: {}",
+                            addr,
                         );
                         Ok::<_, Rejection>(
                             SpspResponder::new(addr, server_secret_clone.clone())
                                 .generate_http_response(),
                         )
                     }
                     Err(err) => {
-                        let msg = format!("Error sending SPSP payment: {}", err);
-                        error!("{}", msg);
+                        error!(?combined, "Appending tracking information produced an invalid address: {}", err);
                         // TODO give a different error message depending on what type of error it is
                         Err(Rejection::from(
                             ApiError::internal_server_error().detail(msg),