Skip to content

feat(node): Avoid OTEL instrumentation for outgoing requests on Node 22+ #17355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mydea wants to merge 20 commits into
base: develop
Choose a base branch
from
+482 −39
Open

feat(node): Avoid OTEL instrumentation for outgoing requests on Node 22+ #17355

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
9c68295
feat(node): Avoid OTEL instrumentation for outgoing requests on Node 22+
mydea Aug 8, 2025
1dad574
Add unit tests for mergeBaggageHeaders
andreiborza Dec 14, 2025
81ba2cf
Bump size limits
andreiborza Dec 14, 2025
52cdf7d
Update `spans` and `createSpansForOutgoingRequests` options docs
andreiborza Dec 14, 2025
a798420
Simplify spansEnabled
andreiborza Dec 14, 2025
e4246f3
Ensure we return the value when patching 'response' events
andreiborza Dec 14, 2025
db7c0e2
Merge branch 'develop' into fn/avoid-otel-http-instrumentation
Lms24 Feb 2, 2026
acf83b6
Merge branch 'develop' into fn/avoid-otel-http-instrumentation
andreiborza Feb 9, 2026
ca643e8
Merge branch 'develop' into fn/avoid-otel-http-instrumentation
andreiborza Feb 18, 2026
c6df808
stricter node version scope to ensure node 22.12.0+
andreiborza Feb 18, 2026
c12e32e
Bump size limit
andreiborza Feb 18, 2026
3ef2033
dedupe getRequestUrl for CLientRequests into a shared getClientReques…
andreiborza Feb 18, 2026
bdcf180
ensure outgoing trace propagation has correct span in context
andreiborza Feb 18, 2026
177b26f
Bump size limits
andreiborza Feb 18, 2026
8c7d414
Forward user instrumentation hooks to SentryHttpInstrumentation
andreiborza Feb 18, 2026
5fc2a89
Fix trace propagation when outgoing request has no parent span
andreiborza Feb 18, 2026
54dae22
Add test asserting consistent trace ID across outgoing requests witho…
andreiborza Feb 18, 2026
4ef95fd
Accept ClientRequest in getRequestUrl type signature
andreiborza Feb 18, 2026
9c09bfd
Remove trace ID consistency assertion from http-no-tracing test
andreiborza Feb 18, 2026
f8e3cad
Fix getContentLength to accept IncomingHttpHeaders
andreiborza Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .size-limit.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ module.exports = [
import: createImport('init'),
ignore: [...builtinModules, ...nodePrefixedBuiltinModules],
gzip: true,
limit: '53 KB',
limit: '57 KB',
},
// Node SDK (ESM)
{
Expand All @@ -326,14 +326,14 @@ module.exports = [
import: createImport('init'),
ignore: [...builtinModules, ...nodePrefixedBuiltinModules],
gzip: true,
limit: '167 KB',
limit: '168 KB',
},
{
name: '@sentry/node - without tracing',
path: 'packages/node/build/esm/index.js',
import: createImport('initWithoutDefaultIntegrations', 'getDefaultIntegrationsWithoutPerformance'),
gzip: true,
limit: '95 KB',
limit: '96 KB',
ignore: [...builtinModules, ...nodePrefixedBuiltinModules],
modifyWebpackConfig: function (config) {
const webpack = require('webpack');
Expand All @@ -356,7 +356,7 @@ module.exports = [
import: createImport('init'),
ignore: [...builtinModules, ...nodePrefixedBuiltinModules],
gzip: true,
limit: '111 KB',
limit: '113 KB',
},
];

Expand Down
Loading
Loading