feat(server-utils): Port SQS, SNS and Lambda aws-sdk extensions with trace propagation by andreiborza · Pull Request #22165 · getsentry/sentry-javascript

andreiborza · 2026-07-09T21:22:15Z

Ports the messaging service extensions from the OTel aws-sdk integration to the orchestrion channel integration and registers them in the service registry:

SQS: producer/consumer span kinds and names, messaging.* attributes, trace propagation into outgoing MessageAttributes (single and batch), and span links from received messages via the propagated headers
SNS: producer span kind/name for Publish, messaging.* and topic ARN attributes, trace propagation into MessageAttributes
Lambda: faas.* attributes for Invoke, trace propagation into the base64 ClientContext (respecting the 3583 byte cap)

Propagation writes Sentry-native sentry-trace/baggage headers derived from the request span instead of OTel propagation.inject, and the SQS receive side reads them back with propagationContextFromHeaders.

Part of #20946

github-actions · 2026-07-09T22:02:45Z

size-limit report 📦

⚠️Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path	Size	% Change	Change
@sentry/browser	27.74 kB	-	-
@sentry/browser - with treeshaking flags	26.19 kB	-	-
@sentry/browser (incl. Tracing)	46.57 kB	-	-
@sentry/browser (incl. Tracing + Span Streaming)	48.36 kB	-	-
@sentry/browser (incl. Tracing, Profiling)	51.34 kB	-	-
@sentry/browser (incl. Tracing, Replay)	85.83 kB	-	-
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	75.46 kB	-	-
@sentry/browser (incl. Tracing, Replay with Canvas)	90.55 kB	-	-
@sentry/browser (incl. Tracing, Replay, Feedback)	103.18 kB	-	-
@sentry/browser (incl. Feedback)	44.92 kB	-	-
@sentry/browser (incl. sendFeedback)	32.54 kB	-	-
@sentry/browser (incl. FeedbackAsync)	37.67 kB	-	-
@sentry/browser (incl. Metrics)	28.84 kB	-	-
@sentry/browser (incl. Logs)	29.07 kB	-	-
@sentry/browser (incl. Metrics & Logs)	29.76 kB	-	-
@sentry/react	29.54 kB	-	-
@sentry/react (incl. Tracing)	48.82 kB	-	-
@sentry/vue	33.17 kB	-	-
@sentry/vue (incl. Tracing)	48.55 kB	-	-
@sentry/svelte	27.77 kB	-	-
CDN Bundle	30.14 kB	-	-
CDN Bundle (incl. Tracing)	48.52 kB	-	-
CDN Bundle (incl. Logs, Metrics)	31.72 kB	-	-
CDN Bundle (incl. Tracing, Logs, Metrics)	49.83 kB	-	-
CDN Bundle (incl. Replay, Logs, Metrics)	70.97 kB	-	-
CDN Bundle (incl. Tracing, Replay)	86.04 kB	-	-
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)	87.33 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback)	91.82 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)	93.09 kB	-	-
CDN Bundle - uncompressed	89.85 kB	-	-
CDN Bundle (incl. Tracing) - uncompressed	146.66 kB	-	-
CDN Bundle (incl. Logs, Metrics) - uncompressed	94.56 kB	-	-
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed	150.64 kB	-	-
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed	219.28 kB	-	-
CDN Bundle (incl. Tracing, Replay) - uncompressed	265.86 kB	-	-
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed	269.82 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	279.56 kB	-	-
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed	283.52 kB	-	-
@sentry/nextjs (client)	51.38 kB	-	-
@sentry/sveltekit (client)	47 kB	-	-
@sentry/core/server	78.71 kB	-	-
@sentry/core/browser	65.08 kB	-	-
@sentry/node-core	63.21 kB	+0.01%	+1 B 🔺
@sentry/node	125.45 kB	-	-
@sentry/node (incl. diagnostics channel injection)	145.67 kB	+2.15%	+3.07 kB 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection)	70.03 kB	-0.01%	-2 B 🔽
@sentry/node/light	51.33 kB	+0.01%	+1 B 🔺
@sentry/node - without tracing	74.7 kB	-0.01%	-2 B 🔽
@sentry/aws-serverless	83.92 kB	-	-
@sentry/cloudflare (withSentry) - minified	182.1 kB	-	-
@sentry/cloudflare (withSentry)	450.9 kB	-	-

View base workflow run

nicohrubec · 2026-07-16T08:44:57Z

+ let spanAttributes: Record<string, unknown> = {};
+ let spanName: string | undefined;
+
+ switch (request.commandName) {


l: this looks a bit weird to me, why use a switch with a single case?

Fair, this has been in the vendored otel instrumentation like this but I'll update to an if

Updated in d8bd9cf

nicohrubec · 2026-07-16T09:00:01Z

+ span.setAttribute(MESSAGING_BATCH_MESSAGE_COUNT, messages.length);
+
+ for (const message of messages) {
+ const headers = extractPropagationHeaders(message);


l: maybe this is a bit easier to read if we refactor the treatment of one message into a function?

Updated in d8bd9cf

nicohrubec · 2026-07-16T09:01:22Z

 /** DynamoDB `db.system` value (an attribute value, not a key, so not covered by conventions). */
 export const DB_SYSTEM_VALUE_DYNAMODB = 'dynamodb';
+
+// Messaging (obsolete OTel convention with no `@sentry/conventions` export, kept for parity)


q: will this change in v11 or do we keep as is?

We can update this in v11, I'll add a todo/task.

Added todo in d8bd9cf

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit d8bd9cf. Configure here.}

…trace propagation

…-link helper, note v11 destination_kind TODO