From 6efce31a94478043abbb2b98ae6f5f4c9d375bad Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Tue, 2 Apr 2024 07:54:24 +1100 Subject: [PATCH] chore(relay): apply log `target` consistently (#4440) The main relay component uses the `relay` target to be more concise whilst emitting logs. This PR fixes a few logs that were missing that. --- rust/relay/src/server.rs | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/rust/relay/src/server.rs b/rust/relay/src/server.rs index 0671a648b..6ce2c3f88 100644 --- a/rust/relay/src/server.rs +++ b/rust/relay/src/server.rs @@ -35,7 +35,7 @@ use stun_codec::rfc8656::attributes::{ }; use stun_codec::rfc8656::errors::{AddressFamilyNotSupported, PeerAddressFamilyMismatch}; use stun_codec::{Message, MessageClass, MessageEncoder, Method, TransactionId}; -use tracing::{field, log, Span}; +use tracing::{field, Span}; use tracing_core::field::display; use uuid::Uuid; @@ -250,16 +250,16 @@ where } // Parsing the bytes failed. Err(client_message::Error::BadChannelData(ref error)) => { - tracing::debug!(%error, "failed to decode channel data") + tracing::debug!(target: "relay", %error, "failed to decode channel data") } Err(client_message::Error::DecodeStun(ref error)) => { - tracing::debug!(%error, "failed to decode stun packet") + tracing::debug!(target: "relay", %error, "failed to decode stun packet") } Err(client_message::Error::UnknownMessageType(t)) => { - tracing::debug!(r#type = %t, "unknown STUN message type") + tracing::debug!(target: "relay", r#type = %t, "unknown STUN message type") } Err(client_message::Error::Eof) => { - tracing::debug!("unexpected EOF while parsing message") + tracing::debug!(target: "relay", "unexpected EOF while parsing message") } }; } @@ -812,7 +812,7 @@ where .value() .parse::() .map_err(|e| { - log::debug!("failed to parse nonce: {e}"); + tracing::debug!(target: "relay", "failed to parse nonce: {e}"); error_response(Unauthorized, request) })?; @@ -955,7 +955,7 @@ where fn delete_allocation(&mut self, id: AllocationId) { let Some(client) = self.clients_by_allocation.remove(&id) else { - tracing::debug!("Unable to delete unknown allocation"); + tracing::debug!(target: "relay", "Unable to delete unknown allocation"); return; };