fix warns

Author: divagant-martian

iroh-net/src/defaults.rs

@@ -134,34 +134,4 @@ pub(crate) mod timeouts {
 
     /// Maximum duration to wait for a netcheck report.
     pub(crate) const NETCHECK_REPORT_TIMEOUT: Duration = Duration::from_secs(10);
-
-    /// The maximum amount of time netcheck will spend gathering a single report.
-    pub(crate) const OVERALL_REPORT_TIMEOUT: Duration = Duration::from_secs(5);
-
-    /// The total time we wait for all the probes.
-    ///
-    /// This includes the STUN, ICMP and HTTPS probes, which will all
-    /// start at different times based on the ProbePlan.
-    pub(crate) const PROBES_TIMEOUT: Duration = Duration::from_secs(3);
-
-    /// How long to await for a captive-portal result.
-    ///
-    /// This delay is chosen so it starts after good-working STUN probes
-    /// would have finished, but not too long so the delay is bearable if
-    /// STUN is blocked.
-    pub(crate) const CAPTIVE_PORTAL_DELAY: Duration = Duration::from_millis(200);
-
-    /// Timeout for captive portal checks
-    ///
-    /// Must be lower than [`OVERALL_REPORT_TIMEOUT`] minus
-    /// [`CAPTIVE_PORTAL_DELAY`].
-    pub(crate) const CAPTIVE_PORTAL_TIMEOUT: Duration = Duration::from_secs(2);
-
-    pub(crate) const DNS_TIMEOUT: Duration = Duration::from_secs(3);
-
-    /// The amount of time we wait for a hairpinned packet to come back.
-    pub(crate) const HAIRPIN_CHECK_TIMEOUT: Duration = Duration::from_millis(100);
-
-    /// Default Pinger timeout
-    pub(crate) const DEFAULT_PINGER_TIMEOUT: Duration = Duration::from_secs(5);
 }

iroh-netcheck/src/dns.rs

@@ -203,13 +203,10 @@ async fn stagger_call<T, F: Fn() -> Fut, Fut: Future<Output = Result<T>>>(
 pub(crate) mod tests {
     use std::net::Ipv6Addr;
     use std::sync::atomic::AtomicUsize;
-    use std::time::Duration;
 
     use once_cell::sync::Lazy;
 
     use super::*;
-    const TIMEOUT: Duration = Duration::from_secs(5);
-    const STAGGERING_DELAYS: &[u64] = &[200, 300];
 
     static DNS_RESOLVER: Lazy<TokioAsyncResolver> =
         Lazy::new(|| create_default_resolver().expect("unable to create DNS resolver"));