async: Do not use heapless on public API.

embedded-nal-async/Cargo.toml

@@ -16,6 +16,5 @@ ip_in_core = []
 
 [dependencies]
 no-std-net = "0.6"
-heapless = "^0.7"
 embedded-nal = { version = "0.7.0", path = "../" }
 embedded-io-async = { version = "0.6.0" }

embedded-nal-async/src/dns.rs

@@ -1,6 +1,5 @@
 use crate::IpAddr;
 use embedded_nal::AddrType;
-use heapless::String;
 
 /// This trait is an extension trait for [`TcpStack`] and [`UdpStack`] for dns
 /// resolutions. It does not handle every DNS record type, but is meant as an
@@ -24,13 +23,22 @@ pub trait Dns {
 		addr_type: AddrType,
 	) -> Result<IpAddr, Self::Error>;
 
-	/// Resolve the hostname of a host, given its ip address
+	/// Resolve the hostname of a host, given its ip address.
+	///
+	/// The result is at the beginning of `result`, the length is returned.
+	///
+	/// If the buffer is too small to hold the domain name, an error should be returned.
 	///
 	/// **Note**: A fully qualified domain name (FQDN), has a maximum length of
-	/// 255 bytes [`rfc1035`]
+	/// 255 bytes according to [`rfc1035`]. Therefore, you can pass a 255-byte long
+	/// buffer to guarantee it'll always be large enough.
 	///
 	/// [`rfc1035`]: https://tools.ietf.org/html/rfc1035
-	async fn get_host_by_address(&self, addr: IpAddr) -> Result<String<256>, Self::Error>;
+	async fn get_host_by_address(
+		&self,
+		addr: IpAddr,
+		result: &mut [u8],
+	) -> Result<usize, Self::Error>;
 }
 
 impl<T: Dns> Dns for &T {
@@ -44,7 +52,11 @@ impl<T: Dns> Dns for &T {
 		T::get_host_by_name(self, host, addr_type).await
 	}
 
-	async fn get_host_by_address(&self, addr: IpAddr) -> Result<String<256>, Self::Error> {
-		T::get_host_by_address(self, addr).await
+	async fn get_host_by_address(
+		&self,
+		addr: IpAddr,
+		result: &mut [u8],
+	) -> Result<usize, Self::Error> {
+		T::get_host_by_address(self, addr, result).await
 	}
 }

embedded-nal-async/src/lib.rs

@@ -9,9 +9,6 @@
 
 mod dns;
 mod stack;
-// Needed by embedded-nal trait implementers who build get_host_by_address results, or by trait
-// users who pass the results on.
-pub use heapless;
 
 #[cfg(feature = "ip_in_core")]
 pub use core::net::{IpAddr, Ipv4Addr, Ipv6Addr, SocketAddr, SocketAddrV4, SocketAddrV6};