Skip to content
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

Jump to bottom

feat(node)!: Collect request sessions via HTTP instrumentation #14658

Draft
wants to merge 30 commits into
base: develop
Choose a base branch
from
Draft

feat(node)!: Collect request sessions via HTTP instrumentation #14658

wants to merge 30 commits into from
+430 −2,159

Conversation

lforst
Copy link
Member

@lforst lforst commented Dec 11, 2024

No description provided.

mydea and others added 9 commits December 6, 2024 10:27
@mydea
ci(v9): Ensure CI runs on v8 & v9 branches (#14604)
7846b24 
In order for us to have size-limit comparison etc, we need to ensure CI
runs on v8 & v9 branches too.
@lforst
feat(vue/nuxt)!: No longer create "update" spans for component trac…
d1ee444 
…king by default (#14602)

Resolves #12851
@lforst
feat(nextjs)!: Remove experimental_captureRequestError (#14607)
26119bf 
Fixes #14302
@lforst
feat(node)!: Collect request sessions via HTTP instrumentation
d7e3626
@lforst
Merge branch 'v9' into lforst-server-sessions
30e2329
@lforst
depr notice
5430723
@mydea
ci(v9): Ensure CI runs on v8 & v9 branches (#14604)
6610144 
In order for us to have size-limit comparison etc, we need to ensure CI
runs on v8 & v9 branches too.
@lforst @mydea
feat(vue/nuxt)!: No longer create "update" spans for component trac…
1648698 
…king by default (#14602)

Resolves #12851
@lforst @mydea
feat(nextjs)!: Remove experimental_captureRequestError (#14607)
d9ee102 
Fixes #14302
lforst and others added 8 commits December 11, 2024 17:48
@lforst
meta(v9): Add v9 migration guide (#14296)
32a1cff
@lforst
feat(node)!: Remove processThreadBreadcrumbIntegration (#14666)
3add6cd 
Resolves #14277
@mydea
feat(node)!: Avoid http spans by default for custom OTEL setups (#14678)
8b81476 
With this PR, the default value for the `spans` option in the
`httpIntegration` is changed to `false`, if `skipOpenTelemetrySetup:
true` is configured. This is what you'd expect as a user, you do not
want Sentry to register any OTEL instrumentation and emit any spans in
this scenario.

Closes #14675
@lforst
Merge remote-tracking branch 'origin/v9' into lforst-server-sessions
c0f9aaa
@lforst
ref!: Don't polyfill optional chaining and nullish coalescing (#14603)
1c9544b
@lforst
Merge branch 'v9' into lforst-server-sessions
b2aed90
@lforst
revert
ac0d55a
@lforst
Hmm
74ac9c3
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 13, 2024 •
edited
Loading

size-limit report 📦

Path Size % Change Change
⛔️ @sentry/browser (max: 23.44 KB) 23.45 KB +0.73% +172 B 🔺
@sentry/browser - with treeshaking flags 22.11 KB +0.7% +156 B 🔺
@sentry/browser (incl. Tracing) 35.95 KB +0.45% +163 B 🔺
@sentry/browser (incl. Tracing, Replay) 73.2 KB +0.24% +174 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 63.56 KB +0.24% +150 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 77.61 KB +0.22% +167 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 89.98 KB +0.16% +147 B 🔺
⛔️ @sentry/browser (incl. Feedback) (max: 40.04 KB) 40.2 KB +0.41% +167 B 🔺
@sentry/browser (incl. sendFeedback) 28.06 KB +0.6% +169 B 🔺
@sentry/browser (incl. FeedbackAsync) 32.84 KB +0.49% +163 B 🔺
@sentry/react 26.14 KB +0.7% +184 B 🔺
@sentry/react (incl. Tracing) 38.77 KB +0.45% +174 B 🔺
@sentry/vue 27.65 KB +0.58% +163 B 🔺
@sentry/vue (incl. Tracing) 37.77 KB +0.4% +153 B 🔺
@sentry/svelte 23.62 KB +0.73% +173 B 🔺
CDN Bundle 24.56 KB +0.54% +135 B 🔺
CDN Bundle (incl. Tracing) 37.58 KB +0.35% +131 B 🔺
CDN Bundle (incl. Tracing, Replay) 72.68 KB +0.2% +142 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 78.07 KB +0.18% +138 B 🔺
CDN Bundle - uncompressed 72.11 KB +0.51% +373 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 111.42 KB +0.34% +379 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 225.47 KB +0.17% +379 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 238.68 KB +0.16% +375 B 🔺
@sentry/nextjs (client) 39.03 KB +0.4% +159 B 🔺
@sentry/sveltekit (client) 36.45 KB +0.44% +160 B 🔺
@sentry/node 162.22 KB -0.33% -546 B 🔽
@sentry/node - without tracing 98.36 KB -0.54% -542 B 🔽
@sentry/aws-serverless 126.09 KB -0.43% -555 B 🔽

View base workflow run

@codecov Codecov
Copy link

codecov bot commented Dec 13, 2024 •
edited
Loading

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
691 1 690 300
View the top 1 failed tests by shortest run time 
public-api/debug/test.ts logs debug messages correctly
Stack Traces | 0.222s run time 
test.ts:7:11 logs debug messages correctly

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

mydea and others added 7 commits December 13, 2024 13:37
@mydea
ci(v9): Ensure CI runs on v8 & v9 branches (#14604)
5e65c87 
In order for us to have size-limit comparison etc, we need to ensure CI
runs on v8 & v9 branches too.
@lforst @mydea
feat(vue/nuxt)!: No longer create "update" spans for component trac…
fac8d97 
…king by default (#14602)

Resolves #12851
@lforst @mydea
feat(nextjs)!: Remove experimental_captureRequestError (#14607)
6772615 
Fixes #14302
@lforst @mydea
meta(v9): Add v9 migration guide (#14296)
6e88c55
@lforst @mydea
feat(node)!: Remove processThreadBreadcrumbIntegration (#14666)
bfd865c 
Resolves #14277
@mydea
feat(node)!: Avoid http spans by default for custom OTEL setups (#14678)
550ceef 
With this PR, the default value for the `spans` option in the
`httpIntegration` is changed to `false`, if `skipOpenTelemetrySetup:
true` is configured. This is what you'd expect as a user, you do not
want Sentry to register any OTEL instrumentation and emit any spans in
this scenario.

Closes #14675
@lforst @mydea
ref!: Don't polyfill optional chaining and nullish coalescing (#14603)
f792e64
@lforst lforst changed the base branch from v9 to develop December 13, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lforst @mydea