refactor(netwatch): Do not use info logging (#8)

This is much to verbose. See also n0-computer/iroh#3071

Author: flub

netwatch/src/netmon/actor.rs

@@ -8,7 +8,7 @@ use anyhow::Result;
 use futures_lite::future::Boxed as BoxFuture;
 use os::{is_interesting_interface, RouteMonitor};
 use tokio::sync::{mpsc, oneshot};
-use tracing::{debug, info, trace, warn};
+use tracing::{debug, trace, warn};
 
 #[cfg(target_os = "android")]
 use super::android as os;
@@ -173,7 +173,7 @@ impl Actor {
     }
 
     async fn handle_potential_change(&mut self, time_jumped: bool) -> Result<()> {
-        info!("potential change");
+        trace!("potential change");
 
         let new_state = State::new().await;
         let old_state = &self.interface_state;

netwatch/src/netmon/linux.rs

@@ -14,7 +14,7 @@ use netlink_packet_route::{address, route, RouteNetlinkMessage};
 use netlink_sys::{AsyncSocket, SocketAddr};
 use rtnetlink::new_connection;
 use tokio::{sync::mpsc, task::JoinHandle};
-use tracing::{info, trace, warn};
+use tracing::{trace, warn};
 
 use super::actor::NetworkMessage;
 use crate::ip::is_link_local;
@@ -78,7 +78,7 @@ impl RouteMonitor {
                         warn!("error reading netlink payload: {:?}", err);
                     }
                     NetlinkPayload::Done(_) => {
-                        info!("done received, exiting");
+                        trace!("done received, exiting");
                         break;
                     }
                     NetlinkPayload::InnerMessage(msg) => match msg {

netwatch/src/udp.rs

@@ -170,7 +170,7 @@ impl UdpSocket {
     /// limiting packets that are read via `recv` from the address specified in
     /// `addr`.
     pub fn connect(&self, addr: SocketAddr) -> io::Result<()> {
-        tracing::info!("connecting to {}", addr);
+        trace!(%addr, "connecting");
         let guard = self.socket.read().unwrap();
         let (socket_tokio, _state) = guard.try_get_connected()?;