Skip to content

ref(opentelemetry): Vendor OTel context suppress helpers - #23030

Draft
nicohrubec wants to merge 1 commit into
developfrom
ref/vendor-suppress-tracing
Draft

ref(opentelemetry): Vendor OTel context suppress helpers#23030
nicohrubec wants to merge 1 commit into
developfrom
ref/vendor-suppress-tracing

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Aug 4, 2026

Copy link
Copy Markdown
Member

Vendors the OTel-context-based suppressTracing/isTracingSuppressed helpers from @opentelemetry/core into a local module, in preparation for dropping that dependency. The vendored copy reuses OTel's exact context key string, since third-party instrumentation like @opentelemetry/instrumentation-http writes suppression to the same key and SentryTracer reads it back to produce non-recording spans.

Fixes #22282

Vendors the OTel-context-based `suppressTracing`/`isTracingSuppressed` helpers
from `@opentelemetry/core` into a local module, so `@sentry/opentelemetry` no
longer imports them from that package. The vendored copy reuses OTel's exact
context key string, since third-party instrumentation (e.g.
`@opentelemetry/instrumentation-http`) writes suppression to the same key and
`SentryTracer` reads it back to produce non-recording spans.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec
nicohrubec force-pushed the ref/vendor-suppress-tracing branch from cbaf7ab to ff34a3b Compare August 4, 2026 19:42
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.83 kB - -
@sentry/browser - with treeshaking flags 28.03 kB - -
@sentry/browser (incl. Tracing) 47.25 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 47.27 kB - -
@sentry/browser (incl. Tracing, Profiling) 52 kB - -
@sentry/browser (incl. Tracing, Replay) 86.59 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 76.01 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 91.28 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 103.95 kB - -
@sentry/browser (incl. Feedback) 47.13 kB - -
@sentry/browser (incl. sendFeedback) 34.67 kB - -
@sentry/browser (incl. FeedbackAsync) 39.78 kB - -
@sentry/browser (incl. Metrics) 30.91 kB - -
@sentry/browser (incl. Logs) 31.14 kB - -
@sentry/browser (incl. Metrics & Logs) 31.81 kB - -
@sentry/react 31.63 kB - -
@sentry/react (incl. Tracing) 49.51 kB - -
@sentry/vue 34.92 kB - -
@sentry/vue (incl. Tracing) 49.24 kB - -
@sentry/svelte 29.86 kB - -
CDN Bundle 31.87 kB - -
CDN Bundle (incl. Tracing) 47.54 kB - -
CDN Bundle (incl. Logs, Metrics) 33.41 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.95 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 72.75 kB - -
CDN Bundle (incl. Tracing, Replay) 85.18 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.5 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 91.01 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.31 kB - -
CDN Bundle - uncompressed 94.62 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.12 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.24 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.1 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 223.94 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.38 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.34 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.08 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.04 kB - -
@sentry/nextjs (client) 52.08 kB - -
@sentry/sveltekit (client) 47.71 kB - -
@sentry/core/server 65.52 kB - -
@sentry/core/browser 51.88 kB - -
@sentry/node 120.23 kB -0.01% -3 B 🔽
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 83.69 kB +0.01% +3 B 🔺
@sentry/aws-serverless 92.36 kB +0.01% +6 B 🔺
@sentry/cloudflare (withSentry) - minified 218.65 kB - -
@sentry/cloudflare (withSentry) 538.85 kB - -

View base workflow run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove suppressTracing / isTracingSuppressed

1 participant