f Remove superfluous SocketAddress newtype

Author: tnull

README.md

@@ -11,8 +11,9 @@ LDK Node is a self-custodial Lightning node in library form. Its central goal is
 The primary abstraction of the library is the [`Node`][api_docs_node], which can be retrieved by setting up and configuring a [`Builder`][api_docs_builder] to your liking and calling one of the `build` methods. `Node` can then be controlled via commands such as `start`, `stop`, `connect_open_channel`, `send_payment`, etc.
 
 ```rust
-use ldk_node::{Builder, SocketAddress};
+use ldk_node::Builder;
 use ldk_node::lightning_invoice::Invoice;
+use ldk_node::lightning::ln::msgs::SocketAddress;
 use ldk_node::bitcoin::secp256k1::PublicKey;
 use ldk_node::bitcoin::Network;
 use std::str::FromStr;

src/builder.rs

@@ -7,7 +7,7 @@ use crate::payment_store::PaymentStore;
 use crate::peer_store::PeerStore;
 use crate::types::{
 	ChainMonitor, ChannelManager, FakeMessageRouter, GossipSync, KeysManager, NetworkGraph,
-	OnionMessenger, PeerManager, SocketAddress,
+	OnionMessenger, PeerManager,
 };
 use crate::wallet::Wallet;
 use crate::LogLevel;
@@ -18,7 +18,7 @@ use crate::{
 
 use lightning::chain::{chainmonitor, BestBlock, Watch};
 use lightning::ln::channelmanager::{self, ChainParameters, ChannelManagerReadArgs};
-use lightning::ln::msgs::RoutingMessageHandler;
+use lightning::ln::msgs::{RoutingMessageHandler, SocketAddress};
 use lightning::ln::peer_handler::{IgnoringMessageHandler, MessageHandler};
 use lightning::routing::router::DefaultRouter;
 use lightning::routing::scoring::{

src/lib.rs

@@ -26,8 +26,9 @@
 //! [`send_payment`], etc.:
 //!
 //! ```no_run
-//! use ldk_node::{Builder, SocketAddress};
+//! use ldk_node::Builder;
 //! use ldk_node::lightning_invoice::Bolt11Invoice;
+//! use ldk_node::lightning::ln::msgs::SocketAddress;
 //! use ldk_node::bitcoin::secp256k1::PublicKey;
 //! use ldk_node::bitcoin::Network;
 //! use std::str::FromStr;
@@ -100,7 +101,6 @@ use error::Error;
 
 pub use event::Event;
 pub use types::ChannelConfig;
-pub use types::SocketAddress;
 
 pub use io::utils::generate_entropy_mnemonic;
 
@@ -126,6 +126,7 @@ use logger::{log_error, log_info, log_trace, FilesystemLogger, Logger};
 
 use lightning::chain::Confirm;
 use lightning::ln::channelmanager::{self, PaymentId, RecipientOnionFields, Retry};
+use lightning::ln::msgs::SocketAddress;
 use lightning::ln::{ChannelId, PaymentHash, PaymentPreimage};
 use lightning::sign::EntropySource;
 
@@ -631,7 +632,7 @@ impl<K: KVStore + Sync + Send + 'static> Node<K> {
 							}
 
 							let addresses =
-								bcast_config.listening_address.iter().cloned().map(|a| a.0).collect();
+								bcast_config.listening_address.iter().cloned().collect();
 							bcast_pm.broadcast_node_announcement([0; 3], [0; 32], addresses);
 
 							let unix_time_secs = SystemTime::now().duration_since(SystemTime::UNIX_EPOCH).unwrap().as_secs();
@@ -1516,7 +1517,7 @@ impl<K: KVStore + Sync + Send + 'static> Node<K> {
 			let stored_peer = self.peer_store.get_peer(&node_id);
 			let stored_addr_opt = stored_peer.as_ref().map(|p| p.address.clone());
 			let address = match (con_addr_opt, stored_addr_opt) {
-				(Some(con_addr), _) => SocketAddress(con_addr),
+				(Some(con_addr), _) => con_addr,
 				(None, Some(stored_addr)) => stored_addr,
 				(None, None) => continue,
 			};

src/types.rs

@@ -4,7 +4,7 @@ use crate::wallet::{Wallet, WalletKeysManager};
 use lightning::chain::chainmonitor;
 use lightning::ln::channelmanager::ChannelDetails as LdkChannelDetails;
 use lightning::ln::msgs::RoutingMessageHandler;
-use lightning::ln::msgs::SocketAddress as LdkSocketAddress;
+use lightning::ln::msgs::SocketAddress;
 use lightning::ln::peer_handler::IgnoringMessageHandler;
 use lightning::ln::ChannelId;
 use lightning::routing::gossip;
@@ -20,9 +20,6 @@ use lightning_transaction_sync::EsploraSyncClient;
 use bitcoin::secp256k1::PublicKey;
 use bitcoin::OutPoint;
 
-use std::fmt::Display;
-use std::net::{Ipv4Addr, Ipv6Addr, SocketAddr, SocketAddrV4, SocketAddrV6, ToSocketAddrs};
-use std::str::FromStr;
 use std::sync::{Arc, Mutex, RwLock};
 
 pub(crate) type ChainMonitor<K> = chainmonitor::ChainMonitor<
@@ -320,106 +317,6 @@ pub struct PeerDetails {
 	pub is_connected: bool,
 }
 
-/// The network address of a Lightning node.
-///
-/// Currently only IPv4, IPv6, and DNS hostnames are supported.
-#[derive(Debug, Clone, PartialEq, Eq)]
-pub struct SocketAddress(pub LdkSocketAddress);
-
-impl Display for SocketAddress {
-	fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-		match self.0 {
-			LdkSocketAddress::TcpIpV4 { addr, port } => {
-				let ip_addr = Ipv4Addr::from(addr);
-				write!(f, "{}:{}", ip_addr, port)
-			}
-			LdkSocketAddress::TcpIpV6 { addr, port } => {
-				let ip_addr = Ipv6Addr::from(addr);
-				write!(f, "[{}]:{}", ip_addr, port)
-			}
-			LdkSocketAddress::Hostname { ref hostname, port } => {
-				write!(f, "{}:{}", hostname.as_str(), port)
-			}
-			LdkSocketAddress::OnionV2(o) => {
-				write!(f, "OnionV2 (unsupported): {:?}", o)
-			}
-			LdkSocketAddress::OnionV3 { ed25519_pubkey, checksum, version, port } => write!(
-				f,
-				"OnionV3 (unsupported): {:?}/{:?}/{:?}/{:?}",
-				ed25519_pubkey, checksum, version, port
-			),
-		}
-	}
-}
-
-impl FromStr for SocketAddress {
-	type Err = ();
-
-	fn from_str(s: &str) -> Result<Self, Self::Err> {
-		Ok(Self(LdkSocketAddress::from_str(s).map_err(|_| ())?))
-	}
-}
-
-impl From<SocketAddr> for SocketAddress {
-	fn from(value: SocketAddr) -> Self {
-		match value {
-			SocketAddr::V4(v4addr) => SocketAddress::from(v4addr),
-			SocketAddr::V6(v6addr) => SocketAddress::from(v6addr),
-		}
-	}
-}
-
-impl From<SocketAddrV4> for SocketAddress {
-	fn from(value: SocketAddrV4) -> Self {
-		Self(LdkSocketAddress::TcpIpV4 { addr: value.ip().octets(), port: value.port() })
-	}
-}
-
-impl From<SocketAddrV6> for SocketAddress {
-	fn from(value: SocketAddrV6) -> Self {
-		Self(LdkSocketAddress::TcpIpV6 { addr: value.ip().octets(), port: value.port() })
-	}
-}
-
-impl ToSocketAddrs for SocketAddress {
-	type Iter = std::option::IntoIter<SocketAddr>;
-
-	fn to_socket_addrs(&self) -> std::io::Result<Self::Iter> {
-		match self.0 {
-			LdkSocketAddress::TcpIpV4 { addr, port } => {
-				let ip_addr = Ipv4Addr::from(addr);
-				(ip_addr, port).to_socket_addrs()
-			}
-			LdkSocketAddress::TcpIpV6 { addr, port } => {
-				let ip_addr = Ipv6Addr::from(addr);
-				(ip_addr, port).to_socket_addrs()
-			}
-			LdkSocketAddress::Hostname { ref hostname, port } => {
-				Ok((hostname.as_str(), port).to_socket_addrs()?.next().into_iter())
-			}
-			LdkSocketAddress::OnionV2(..) => {
-				Err(std::io::Error::from(std::io::ErrorKind::Unsupported))
-			}
-			LdkSocketAddress::OnionV3 { .. } => {
-				Err(std::io::Error::from(std::io::ErrorKind::Unsupported))
-			}
-		}
-	}
-}
-
-impl Writeable for SocketAddress {
-	fn write<W: lightning::util::ser::Writer>(&self, writer: &mut W) -> Result<(), std::io::Error> {
-		self.0.write(writer)
-	}
-}
-
-impl Readable for SocketAddress {
-	fn read<R: std::io::Read>(reader: &mut R) -> Result<Self, lightning::ln::msgs::DecodeError> {
-		let addr: LdkSocketAddress = Readable::read(reader)?;
-		Ok(Self(addr))
-	}
-}
-
 /// Options which apply on a per-channel basis.
 ///
 /// See documentation of [`LdkChannelConfig`] for details.