De-/Serialize addresses assuming their network isn't bogus

.. as we have no real way to check the network at the point of
deserialzation, and we want to handle `bitcoin::Addresses`, not uncheck
addresses, in particular when it comes to serialization.

src/lsps0/ser.rs

@@ -654,3 +654,29 @@ pub(crate) mod u32_fee_rate {
 		Ok(FeeRate::from_sat_per_kwu(fee_rate_sat_kwu as u64))
 	}
 }
+
+pub(crate) mod unchecked_address {
+	use crate::prelude::{String, ToString};
+	use core::str::FromStr;
+	use serde::de::Unexpected;
+	use serde::{Deserialize, Deserializer, Serializer};
+
+	pub(crate) fn serialize<S>(x: &bitcoin::Address, s: S) -> Result<S::Ok, S::Error>
+	where
+		S: Serializer,
+	{
+		s.serialize_str(&x.to_string())
+	}
+
+	pub(crate) fn deserialize<'de, D>(deserializer: D) -> Result<bitcoin::Address, D::Error>
+	where
+		D: Deserializer<'de>,
+	{
+		let buf = String::deserialize(deserializer)?;
+
+		let parsed_addr = bitcoin::Address::from_str(&buf).map_err(|_| {
+			serde::de::Error::invalid_value(Unexpected::Str(&buf), &"invalid address string")
+		})?;
+		Ok(parsed_addr.assume_checked())
+	}
+}

src/lsps1/msgs.rs

@@ -1,6 +1,8 @@
 //! Message, request, and other primitive types used to implement LSPS1.
 
-use crate::lsps0::ser::{string_amount, u32_fee_rate, LSPSMessage, RequestId, ResponseError};
+use crate::lsps0::ser::{
+	string_amount, u32_fee_rate, unchecked_address, LSPSMessage, RequestId, ResponseError,
+};
 
 use crate::prelude::String;
 
@@ -182,7 +184,8 @@ pub struct OnchainPaymentInfo {
 	pub order_total_sat: u64,
 	/// An on-chain address the client can send [`Self::order_total_sat`] to to have the channel
 	/// opened.
-	pub address: Address<NetworkUnchecked>,
+	#[serde(with = "unchecked_address")]
+	pub address: Address,
 	/// The minimum number of block confirmations that are required for the on-chain payment to be
 	/// considered confirmed.
 	pub min_onchain_payment_confirmations: Option<u16>,