github-personal/firezone
1
0
Fork 0
mirror of https://github.com/outbackdingo/firezone.git synced 2026-01-27 10:18:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(telemetry): remove IGNORED_TARGETS (#7365)

Browse Source 
Sentry allows us to do this kind of filtering on the server-side, we can
thus remove this complexity from our codebase.
This commit is contained in:
Thomas Eizinger
2024-11-18 18:17:30 +00:00
committed by GitHub
parent de35bb067e
commit 11dd0c42ca
1 changed files with 5 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
rust/logging/src/lib.rs
View File

@@ -106,24 +106,12 @@ pub fn sentry_layer<S>() -> impl Layer<S> + Send + Sync
where
S: Subscriber + for<'a> LookupSpan<'a>,
{
const IGNORED_TARGETS: &[IgnoredEvent] = &[IgnoredEvent::warn(
"tao::platform_impl::platform::event_loop::runner",
)];
sentry_tracing::layer()
.event_filter(move |md| {
let level = *md.level();
if IGNORED_TARGETS.contains(&IgnoredEvent::new(*md.level(), md.target())) {
return EventFilter::Ignore;
}
match level {
tracing::Level::ERROR | tracing::Level::WARN => EventFilter::Exception,
tracing::Level::INFO | tracing::Level::DEBUG => EventFilter::Breadcrumb,
tracing::Level::TRACE if md.target() == TELEMETRY_TARGET => EventFilter::Event,
_ => EventFilter::Ignore,
}
.event_filter(move |md| match *md.level() {
tracing::Level::ERROR | tracing::Level::WARN => EventFilter::Exception,
tracing::Level::INFO | tracing::Level::DEBUG => EventFilter::Breadcrumb,
tracing::Level::TRACE if md.target() == TELEMETRY_TARGET => EventFilter::Event,
_ => EventFilter::Ignore,
})
.span_filter(|md| *md.level() == tracing::Level::TRACE && md.target() == TELEMETRY_TARGET)
.enable_span_attributes()
@@ -169,22 +157,3 @@ pub mod __export {
pub use rand;
pub use tracing;
}
#[derive(Debug, PartialEq)]
struct IgnoredEvent<'a> {
level: tracing::Level,
target: &'a str,
}
impl<'a> IgnoredEvent<'a> {
fn new(level: tracing::Level, target: &'a str) -> Self {
Self { level, target }
}
const fn warn(target: &'static str) -> Self {
Self {
target,
level: tracing::Level::WARN,
}
}
}