Patch for libtelio

Reduce log levels from error and warn to debug.
These changes are copied from 0be935b commit.

Author: dfetti

bin/src/named.rs

@@ -51,7 +51,7 @@ use tokio::{
     net::{TcpListener, UdpSocket},
     runtime,
 };
-use tracing::{debug, error, info, warn, Event, Subscriber};
+use tracing::{debug, info, Event, Subscriber};
 use tracing_subscriber::{
     fmt::{format, FmtContext, FormatEvent, FormatFields, FormattedFields},
     layer::SubscriberExt,
@@ -157,15 +157,15 @@ async fn load_zone(
     let is_dnssec_enabled = zone_config.is_dnssec_enabled();
 
     if zone_config.is_update_allowed() {
-        warn!("allow_update is deprecated in [[zones]] section, it belongs in [[zones.stores]]");
+        debug!("allow_update is deprecated in [[zones]] section, it belongs in [[zones.stores]]");
     }
 
     // load the zone
     let authority: Box<dyn AuthorityObject> = match zone_config.stores {
         #[cfg(feature = "sqlite")]
         Some(StoreConfig::Sqlite(ref config)) => {
             if zone_path.is_some() {
-                warn!("ignoring [[zones.file]] instead using [[zones.stores.zone_file_path]]");
+                debug!("ignoring [[zones.file]] instead using [[zones.stores.zone_file_path]]");
             }
 
             let mut authority = SqliteAuthority::try_from_config(
@@ -184,7 +184,7 @@ async fn load_zone(
         }
         Some(StoreConfig::File(ref config)) => {
             if zone_path.is_some() {
-                warn!("ignoring [[zones.file]] instead using [[zones.stores.zone_file_path]]");
+                debug!("ignoring [[zones.file]] instead using [[zones.stores.zone_file_path]]");
             }
 
             let mut authority = FileAuthority::try_from_config(
@@ -215,7 +215,7 @@ async fn load_zone(
         }
         #[cfg(feature = "sqlite")]
         None if zone_config.is_update_allowed() => {
-            warn!(
+            debug!(
                 "using deprecated SQLite load configuration, please move to [[zones.stores]] form"
             );
             let zone_file_path = zone_path.ok_or("file is a necessary parameter of zone_config")?;
@@ -559,7 +559,7 @@ fn main() {
                 e
             );
 
-            error!("{}", error_msg);
+            debug!("{}", error_msg);
             panic!("{}", error_msg);
         }
     };
@@ -586,7 +586,7 @@ fn config_tls(
         .collect();
 
     if tls_sockaddrs.is_empty() {
-        warn!("a tls certificate was specified, but no TLS addresses configured to listen on");
+        debug!("a tls certificate was specified, but no TLS addresses configured to listen on");
     }
 
     for tls_listener in &tls_sockaddrs {
@@ -639,7 +639,7 @@ fn config_https(
         .collect();
 
     if https_sockaddrs.is_empty() {
-        warn!("a tls certificate was specified, but no HTTPS addresses configured to listen on");
+        debug!("a tls certificate was specified, but no HTTPS addresses configured to listen on");
     }
 
     for https_listener in &https_sockaddrs {
@@ -698,7 +698,7 @@ fn config_quic(
         .collect();
 
     if quic_sockaddrs.is_empty() {
-        warn!("a tls certificate was specified, but no QUIC addresses configured to listen on");
+        debug!("a tls certificate was specified, but no QUIC addresses configured to listen on");
     }
 
     for quic_listener in &quic_sockaddrs {

bin/tests/server_harness/mod.rs

@@ -14,7 +14,7 @@ use std::time::*;
 use regex::Regex;
 use tokio::runtime::Runtime;
 
-use tracing::{info, warn};
+use tracing::{debug, info};
 use trust_dns_client::client::*;
 use trust_dns_client::proto::xfer::DnsResponse;
 use trust_dns_client::rr::*;
@@ -84,7 +84,7 @@ where
 
                 let mut named = named_killer.lock().unwrap();
                 if let Err(e) = named.kill() {
-                    warn!("warning: failed to kill named: {:?}", e);
+                    debug!("warning: failed to kill named: {:?}", e);
                 }
             };
 

crates/client/src/serialize/txt/rdata_parsers/caa.rs

@@ -16,7 +16,7 @@
 
 //! mail exchange, email, record
 
-use tracing::warn;
+use tracing::debug;
 
 use crate::proto::rr::rdata::caa;
 use crate::proto::rr::rdata::caa::{Property, Value};
@@ -60,7 +60,7 @@ pub(crate) fn parse<'i, I: Iterator<Item = &'i str>>(mut tokens: I) -> ParseResu
     let issuer_critical = {
         let flags = flags_str.parse::<u8>()?;
         if flags & 0b0111_1111 != 0 {
-            warn!("unexpected flag values in caa (0 or 128): {}", flags);
+            debug!("unexpected flag values in caa (0 or 128): {}", flags);
         }
 
         flags & 0b1000_0000 != 0
@@ -71,7 +71,7 @@ pub(crate) fn parse<'i, I: Iterator<Item = &'i str>>(mut tokens: I) -> ParseResu
         // unnecessary clone
         let tag = Property::from(tag_str.to_string());
         if tag.is_unknown() {
-            warn!("unknown tag found for caa: {:?}", tag);
+            debug!("unknown tag found for caa: {:?}", tag);
         }
         tag
     };

crates/proto/src/https/https_client_stream.rs

@@ -26,7 +26,7 @@ use rustls::ClientConfig;
 use tokio_rustls::{
     client::TlsStream as TokioTlsClientStream, Connect as TokioTlsConnect, TlsConnector,
 };
-use tracing::{debug, warn};
+use tracing::debug;
 
 use crate::error::ProtoError;
 use crate::iocompat::AsyncIoStdAsTokio;
@@ -476,7 +476,7 @@ where
                     debug!("h2 connection established to: {}", name_server);
                     tokio::spawn(
                         connection
-                            .map_err(|e| warn!("h2 connection failed: {}", e))
+                            .map_err(|e| debug!("h2 connection failed: {}", e))
                             .map(|_: Result<(), ()>| ()),
                     );
 

crates/proto/src/op/message.rs

@@ -9,7 +9,7 @@
 
 use std::{fmt, iter, mem, ops::Deref, sync::Arc};
 
-use tracing::{debug, warn};
+use tracing::debug;
 
 use crate::{
     error::*,
@@ -972,7 +972,7 @@ where
         additional_count.0 += count.0;
         additional_count.1 |= count.1;
     } else if header.response_code().high() > 0 {
-        warn!(
+        debug!(
             "response code: {} for request: {} requires EDNS but none available",
             header.response_code(),
             header.id()

crates/proto/src/rr/dnssec/rdata/dnskey.rs

@@ -249,7 +249,7 @@ impl DNSKEY {
                 .emit(&mut encoder)
                 .and_then(|_| emit(&mut encoder, self))
             {
-                tracing::warn!("error serializing dnskey: {}", e);
+                tracing::debug!("error serializing dnskey: {}", e);
                 return Err(format!("error serializing dnskey: {}", e).into());
             }
         }

crates/proto/src/rr/dnssec/supported_algorithm.rs

@@ -23,7 +23,7 @@ use std::fmt::{Display, Formatter};
 #[cfg(feature = "serde-config")]
 use serde::{Deserialize, Serialize};
 
-use tracing::warn;
+use tracing::debug;
 
 use crate::error::*;
 use crate::rr::dnssec::Algorithm;
@@ -149,7 +149,7 @@ impl<'a> From<&'a [u8]> for SupportedAlgorithms {
 
         for a in values.iter().map(|i| Algorithm::from_u8(*i)) {
             match a {
-                Algorithm::Unknown(v) => warn!("unrecognized algorithm: {}", v),
+                Algorithm::Unknown(v) => debug!("unrecognized algorithm: {}", v),
                 a => supported.set(a),
             }
         }

crates/proto/src/rr/rdata/opt.rs

@@ -22,7 +22,7 @@ use std::collections::HashMap;
 #[cfg(feature = "serde-config")]
 use serde::{Deserialize, Serialize};
 
-use tracing::warn;
+use tracing::debug;
 
 use crate::error::*;
 use crate::serialize::binary::*;
@@ -285,7 +285,7 @@ pub fn read(decoder: &mut BinDecoder<'_>, rdata_length: Restrict<u16>) -> ProtoR
     if state != OptReadState::ReadCode {
         // there was some problem parsing the data for the options, ignoring them
         // TODO: should we ignore all of the EDNS data in this case?
-        warn!("incomplete or poorly formatted EDNS options: {:?}", state);
+        debug!("incomplete or poorly formatted EDNS options: {:?}", state);
         options.clear();
     }
 

crates/proto/src/rr/record_data.rs

@@ -18,7 +18,7 @@ use std::net::{IpAddr, Ipv4Addr, Ipv6Addr};
 use serde::{Deserialize, Serialize};
 
 use enum_as_inner::EnumAsInner;
-use tracing::{trace, warn};
+use tracing::{debug, trace};
 
 use super::domain::Name;
 use super::rdata;
@@ -698,7 +698,7 @@ impl RData {
         {
             let mut encoder: BinEncoder<'_> = BinEncoder::new(&mut buf);
             self.emit(&mut encoder).unwrap_or_else(|_| {
-                warn!("could not encode RDATA: {:?}", self);
+                debug!("could not encode RDATA: {:?}", self);
             });
         }
         buf

crates/proto/src/rr/rr_set.rs

@@ -8,7 +8,7 @@ use std::iter::Chain;
 use std::slice::Iter;
 use std::vec;
 
-use tracing::{info, warn};
+use tracing::{debug, info};
 
 use crate::rr::{DNSClass, Name, RData, Record, RecordType};
 
@@ -312,7 +312,7 @@ impl RecordSet {
                             }
                         }
                         rdata => {
-                            warn!("wrong rdata: {:?}, expected SOA", rdata);
+                            debug!("wrong rdata: {:?}, expected SOA", rdata);
                             return false;
                         }
                     }