Skip to content

Commit

Permalink
remove Context argument from breadcrumb_from_event
Browse files Browse the repository at this point in the history
  • Loading branch information
tsionyx committed Dec 5, 2023
1 parent 7ba8dee commit a112a00
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
5 changes: 1 addition & 4 deletions sentry-tracing/src/converters.rs
Original file line number Diff line number Diff line change
Expand Up @@ -147,10 +147,7 @@ impl Visit for FieldVisitor {
}

/// Creates a [`Breadcrumb`] from a given [`tracing_core::Event`]
pub fn breadcrumb_from_event<'context, S>(
event: &tracing_core::Event,
_ctx: impl Into<Option<Context<'context, S>>>,
) -> Breadcrumb
pub fn breadcrumb_from_event<S>(event: &tracing_core::Event) -> Breadcrumb
where
S: Subscriber + for<'a> LookupSpan<'a>,
{
Expand Down
2 changes: 1 addition & 1 deletion sentry-tracing/src/layer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ where
match (self.event_filter)(event.metadata()) {
EventFilter::Ignore => EventMapping::Ignore,
EventFilter::Breadcrumb => {
EventMapping::Breadcrumb(breadcrumb_from_event(event, span_ctx))
EventMapping::Breadcrumb(breadcrumb_from_event::<S>(event))
}
EventFilter::Event => EventMapping::Event(event_from_event(event, span_ctx)),
EventFilter::Exception => {
Expand Down

0 comments on commit a112a00

Please sign in to comment.