fix(examples/coap): Various fixes (#485)

examples/coap/Cargo.toml

@@ -45,5 +45,5 @@ scroll-ring = "0.1.1"
 [features]
 default = ["proto-ipv4"] # shame
 # actually embedded-nal features, we have to match them here while developing udp_nal in here
-proto-ipv4 = []
-proto-ipv6 = []
+proto-ipv4 = ["embassy-net/proto-ipv4"]
+proto-ipv6 = ["embassy-net/proto-ipv6"]

examples/coap/src/main.rs

@@ -6,7 +6,6 @@
 use riot_rs::{debug::log::*, network};
 
 use embassy_net::udp::{PacketMetadata, UdpSocket};
-use embedded_nal_coap::TransportError;
 
 // Moving work from https://github.com/embassy-rs/embassy/pull/2519 in here for the time being
 mod udp_nal;
@@ -95,7 +94,7 @@ where
         R,
         );
 
-    let mut handler = coap_message_demos::full_application_tree(log)
+    let handler = coap_message_demos::full_application_tree(log)
         .at(
             &["stdout"],
             coap_scroll_ring_server::BufferHandler::new(&buffer),

examples/coap/src/udp_nal/mod.rs

@@ -76,7 +76,7 @@ impl<'a> ConnectedUdp<'a> {
     /// unconnected.
     pub async fn connect(socket: udp::UdpSocket<'a>, /*, ... */) -> Result<Self, udp::BindError> {
         // This is really just a copy of the provided `embedded_nal::udp::UdpStack::connect` method
-        todo!()
+        todo!("use {:p}", &socket)
     }
 }
 

examples/coap/src/udp_nal/util.rs

@@ -38,7 +38,8 @@ pub(super) fn sockaddr_smol2nal(endpoint: IpEndpoint) -> nal::SocketAddr {
         }
         #[cfg(feature = "proto-ipv6")]
         IpAddress::Ipv6(addr) => {
-            embedded_nal_async::SocketAddrV6::new(addr.0.into(), endpoint.port).into()
+            // FIXME: Where is smoltcp's zone identifier?
+            embedded_nal_async::SocketAddrV6::new(addr.0.into(), endpoint.port, 0, 0).into()
         }
     }
 }