Merge pull request #920 from NordSecurity/LLT-5747_lower_session_keep…

…er_warning_verbocity

Lower session keeper and DNS logging verbocity

crates/telio-dns/src/forward.rs

@@ -220,9 +220,22 @@ impl Authority for ForwardAuthority {
         telio_log_debug!("forwarding lookup: {} {}", name, rtype);
         let resolve = self.resolver.lookup(name.clone(), rtype).await;
 
-        if resolve.is_err() {
-            telio_log_warn!("DNS name resolution failed with {:?}", resolve);
-        }
+        // Log DNS failures
+        match resolve {
+            // Some errors are not really relevant, and happens in a wild often.
+            // For example: no IPs associated with domain especially for AAAA queries
+            //
+            // Log such errors with lower logging level
+            Err(ref e)
+                if matches!(e.kind(), ResolveErrorKind::NoRecordsFound { .. })
+                    && (rtype == RecordType::AAAA || rtype == RecordType::SOA) =>
+            {
+                telio_log_debug!("DNS name resolution failed with {:?}", e);
+            }
+
+            Err(ref e) => telio_log_warn!("DNS name resolution failed with {:?}", e),
+            Ok(_) => (),
+        };
 
         resolve
             .map(ForwardLookup)

src/device/wg_controller.rs

@@ -398,7 +398,7 @@ async fn consolidate_wg_peers<
                     telio_log_warn!("Peer {:?} has no ip address", key);
                 }
             }
-            (None, _) => telio_log_warn!("The session keeper is missing!"),
+            (None, _) => telio_log_debug!("The session keeper is missing!"),
             _ => (),
         }
 
@@ -455,7 +455,7 @@ async fn consolidate_wg_peers<
                 }
             }
             (Some(_), _, _) => (),
-            (None, _, _) => telio_log_warn!("The session keeper is missing!"),
+            (None, _, _) => telio_log_debug!("The session keeper is missing!"),
         }
 
         let is_actual_peer_proxying = is_peer_proxying(actual_peer, &proxy_endpoints);