fixup: Use display instead of debug on WebhookNotification

Author: martinsaposnic

lightning-liquidity/src/lsps5/msgs.rs

@@ -461,7 +461,7 @@ pub enum WebhookNotificationMethod {
 }
 
 /// Webhook notification payload.
-#[derive(Clone, PartialEq, Eq)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 pub struct WebhookNotification {
 	/// Notification method with parameters.
 	pub method: WebhookNotificationMethod,
@@ -499,13 +499,11 @@ impl WebhookNotification {
 	}
 }
 
-impl fmt::Debug for WebhookNotification {
+impl fmt::Display for WebhookNotification {
 	fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
 		match serde_json::to_string(self) {
 			Ok(json) => f.write_str(&json),
-			// WebhookNotification is well defined and does not contain arbitrary data, so
-			// this case should never happen.
-			Err(_) => f.write_str("<failed-to-serialize>"),
+			Err(_) => Err(fmt::Error),
 		}
 	}
 }

lightning-liquidity/src/lsps5/service.rs

@@ -435,7 +435,7 @@ where
 
 	fn sign_notification(&self, body: &WebhookNotification, timestamp: &LSPSDateTime) -> String {
 		let message = format!(
-			"LSPS5: DO NOT SIGN THIS MESSAGE MANUALLY: LSP: At {} I notify {:?}",
+			"LSPS5: DO NOT SIGN THIS MESSAGE MANUALLY: LSP: At {} I notify {}",
 			timestamp.to_rfc3339(),
 			body
 		);

lightning-liquidity/src/lsps5/validator.rs

@@ -101,7 +101,7 @@ where
 		}
 
 		let message = format!(
-			"LSPS5: DO NOT SIGN THIS MESSAGE MANUALLY: LSP: At {} I notify {:?}",
+			"LSPS5: DO NOT SIGN THIS MESSAGE MANUALLY: LSP: At {} I notify {}",
 			signature_timestamp.to_rfc3339(),
 			notification
 		);