Skip to content

Commit

Permalink
Don't sentry breadcrumb trace-level messages (#32901)
Browse files Browse the repository at this point in the history
They are extremely verbose and use a lot of memory.
Notably aws-smithy traces the entire request on an S3 upload.

from inside orchestrator.rs in aws-smithy-runtime
```
        trace!(request = ?request, "transmitting request");
```

GitOrigin-RevId: ead23b8ce867f52b609a71c566a6beea7469aebc
  • Loading branch information
nipunn1313 authored and Convex, Inc. committed Jan 8, 2025
1 parent 1af6eab commit 77456d4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion crates/cmd_util/src/env.rs
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ where
layers.push(format_layer);
let sentry_layer = sentry_tracing::layer()
.event_filter(|md| match md.level() {
&tracing::Level::DEBUG => EventFilter::Ignore,
&tracing::Level::DEBUG | &tracing::Level::TRACE => EventFilter::Ignore,
_ => EventFilter::Breadcrumb,
})
.span_filter(|_md| false);
Expand Down

0 comments on commit 77456d4

Please sign in to comment.