Skip to content

Commit 20977df

Browse files
chargomechargome
committed
Merge branch 'develop' into cg-metrics-flag
2 parents 218425c + ad870cf commit 20977df

File tree

192 files changed

+851
-671
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

192 files changed

+851
-671
lines changed

‎.size-limit.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ module.exports = [
197197
path: createCDNPath('bundle.tracing.replay.min.js'),
198198
gzip: false,
199199
brotli: false,
200-
limit: '240 KB',
200+
limit: '245 KB',
201201
},
202202
{
203203
name: 'CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed',

‎CHANGELOG.md‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,37 @@
44

55
- "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
66

7+
## 10.23.0
8+
9+
- feat(core): Send `user-agent` header with envelope requests in server SDKs ([#17929](https://github.com/getsentry/sentry-javascript/pull/17929))
10+
- feat(browser): Limit transport buffer size ([#18046](https://github.com/getsentry/sentry-javascript/pull/18046))
11+
- feat(core): Remove default value of `maxValueLength: 250` ([#18043](https://github.com/getsentry/sentry-javascript/pull/18043))
12+
- feat(react-router): Align options with shared build time options type ([#18014](https://github.com/getsentry/sentry-javascript/pull/18014))
13+
- fix(browser-utils): cache element names for INP ([#18052](https://github.com/getsentry/sentry-javascript/pull/18052))
14+
- fix(browser): Capture unhandled rejection errors for web worker integration ([#18054](https://github.com/getsentry/sentry-javascript/pull/18054))
15+
- fix(cloudflare): Ensure types for cloudflare handlers ([#18064](https://github.com/getsentry/sentry-javascript/pull/18064))
16+
- fix(nextjs): Update proxy template wrapping ([#18086](https://github.com/getsentry/sentry-javascript/pull/18086))
17+
- fix(nuxt): Added top-level fallback exports ([#18083](https://github.com/getsentry/sentry-javascript/pull/18083))
18+
- fix(nuxt): check for H3 error cause before re-capturing ([#18035](https://github.com/getsentry/sentry-javascript/pull/18035))
19+
- fix(replay): Linked errors not resetting session id ([#17854](https://github.com/getsentry/sentry-javascript/pull/17854))
20+
- fix(tracemetrics): Bump metrics buffer to 1k ([#18039](https://github.com/getsentry/sentry-javascript/pull/18039))
21+
- fix(vue): Make `options` parameter optional on `attachErrorHandler` ([#18072](https://github.com/getsentry/sentry-javascript/pull/18072))
22+
- ref(core): Set span status `internal_error` instead of `unknown_error` ([#17909](https://github.com/getsentry/sentry-javascript/pull/17909))
23+
24+
<details>
25+
<summary> <strong>Internal Changes</strong> </summary>
26+
27+
- fix(tests): un-override nitro dep version for nuxt-3 test ([#18056](https://github.com/getsentry/sentry-javascript/pull/18056))
28+
- fix(e2e): Add p-map override to fix React Router 7 test builds ([#18068](https://github.com/getsentry/sentry-javascript/pull/18068))
29+
- feat: Add a note to save changes before starting ([#17987](https://github.com/getsentry/sentry-javascript/pull/17987))
30+
- test(browser): Add test for INP target name after navigation or DOM changes ([#18033](https://github.com/getsentry/sentry-javascript/pull/18033))
31+
- chore: Add external contributor to CHANGELOG.md ([#18032](https://github.com/getsentry/sentry-javascript/pull/18032))
32+
- chore(aws-serverless): Fix typo in timeout warning function name ([#18031](https://github.com/getsentry/sentry-javascript/pull/18031))
33+
- chore(browser): upgrade fake-indexeddb to v6 ([#17975](https://github.com/getsentry/sentry-javascript/pull/17975))
34+
- chore(tests): pass test flags through to the test command ([#18062](https://github.com/getsentry/sentry-javascript/pull/18062))
35+
36+
</details>
37+
738
Work in this release was contributed by @hanseo0507. Thank you for your contribution!
839

940
## 10.22.0

‎dev-packages/browser-integration-tests/.eslintrc.js‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@ module.exports = {
33
browser: true,
44
node: true,
55
},
6-
extends: ['../../.eslintrc.js'],
6+
// todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
7+
extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
8+
plugins: ['regexp'],
79
ignorePatterns: [
810
'suites/**/subject.js',
911
'suites/**/dist/*',

‎dev-packages/browser-integration-tests/package.json‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry-internal/browser-integration-tests",
3-
"version": "10.22.0",
3+
"version": "10.23.0",
44
"main": "index.js",
55
"license": "MIT",
66
"engines": {
@@ -43,7 +43,7 @@
4343
"@babel/preset-typescript": "^7.16.7",
4444
"@playwright/test": "~1.53.2",
4545
"@sentry-internal/rrweb": "2.34.0",
46-
"@sentry/browser": "10.22.0",
46+
"@sentry/browser": "10.23.0",
4747
"@supabase/supabase-js": "2.49.3",
4848
"axios": "^1.12.2",
4949
"babel-loader": "^8.2.2",
@@ -54,6 +54,7 @@
5454
"devDependencies": {
5555
"@types/glob": "8.0.0",
5656
"@types/node": "^18.19.1",
57+
"eslint-plugin-regexp": "^1.15.0",
5758
"glob": "8.0.3"
5859
},
5960
"volta": {

‎dev-packages/browser-integration-tests/suites/manual-client/browser-context/test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ sentryTest('allows to setup a client manually & capture exceptions', async ({ ge
4646
},
4747
},
4848
contexts: {
49-
trace: { trace_id: expect.stringMatching(/[a-f0-9]{32}/), span_id: expect.stringMatching(/[a-f0-9]{16}/) },
49+
trace: { trace_id: expect.stringMatching(/[a-f\d]{32}/), span_id: expect.stringMatching(/[a-f\d]{16}/) },
5050
},
5151
});
5252
});

‎dev-packages/browser-integration-tests/suites/profiling/traceLifecycleMode_multiple-chunks/test.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,9 +58,9 @@ sentryTest(
5858

5959
// Required profile metadata (Sample Format V2)
6060
expect(typeof envelopeItemPayload1.profiler_id).toBe('string');
61-
expect(envelopeItemPayload1.profiler_id).toMatch(/^[a-f0-9]{32}$/);
61+
expect(envelopeItemPayload1.profiler_id).toMatch(/^[a-f\d]{32}$/);
6262
expect(typeof envelopeItemPayload1.chunk_id).toBe('string');
63-
expect(envelopeItemPayload1.chunk_id).toMatch(/^[a-f0-9]{32}$/);
63+
expect(envelopeItemPayload1.chunk_id).toMatch(/^[a-f\d]{32}$/);
6464
expect(envelopeItemPayload1.client_sdk).toBeDefined();
6565
expect(typeof envelopeItemPayload1.client_sdk.name).toBe('string');
6666
expect(typeof envelopeItemPayload1.client_sdk.version).toBe('string');
@@ -170,9 +170,9 @@ sentryTest(
170170
// Required profile metadata (Sample Format V2)
171171
// https://develop.sentry.dev/sdk/telemetry/profiles/sample-format-v2/
172172
expect(typeof envelopeItemPayload2.profiler_id).toBe('string');
173-
expect(envelopeItemPayload2.profiler_id).toMatch(/^[a-f0-9]{32}$/);
173+
expect(envelopeItemPayload2.profiler_id).toMatch(/^[a-f\d]{32}$/);
174174
expect(typeof envelopeItemPayload2.chunk_id).toBe('string');
175-
expect(envelopeItemPayload2.chunk_id).toMatch(/^[a-f0-9]{32}$/);
175+
expect(envelopeItemPayload2.chunk_id).toMatch(/^[a-f\d]{32}$/);
176176
expect(envelopeItemPayload2.client_sdk).toBeDefined();
177177
expect(typeof envelopeItemPayload2.client_sdk.name).toBe('string');
178178
expect(typeof envelopeItemPayload2.client_sdk.version).toBe('string');

‎dev-packages/browser-integration-tests/suites/profiling/traceLifecycleMode_overlapping-spans/test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ sentryTest(
6060
// Required profile metadata (Sample Format V2)
6161
// https://develop.sentry.dev/sdk/telemetry/profiles/sample-format-v2/
6262
expect(typeof envelopeItemPayload.profiler_id).toBe('string');
63-
expect(envelopeItemPayload.profiler_id).toMatch(/^[a-f0-9]{32}$/);
63+
expect(envelopeItemPayload.profiler_id).toMatch(/^[a-f\d]{32}$/);
6464
expect(typeof envelopeItemPayload.chunk_id).toBe('string');
65-
expect(envelopeItemPayload.chunk_id).toMatch(/^[a-f0-9]{32}$/);
65+
expect(envelopeItemPayload.chunk_id).toMatch(/^[a-f\d]{32}$/);
6666
expect(envelopeItemPayload.client_sdk).toBeDefined();
6767
expect(typeof envelopeItemPayload.client_sdk.name).toBe('string');
6868
expect(typeof envelopeItemPayload.client_sdk.version).toBe('string');
@@ -175,7 +175,7 @@ sentryTest('attaches thread data to child spans (trace mode)', async ({ page, ge
175175
const profilerId = rootSpan?.contexts?.profile?.profiler_id as string | undefined;
176176
expect(typeof profilerId).toBe('string');
177177

178-
expect(profilerId).toMatch(/^[a-f0-9]{32}$/);
178+
expect(profilerId).toMatch(/^[a-f\d]{32}$/);
179179

180180
const spans = (rootSpan?.spans ?? []) as Array<{ data?: Record<string, unknown> }>;
181181
expect(spans.length).toBeGreaterThan(0);

‎dev-packages/browser-integration-tests/suites/public-api/startSpan/standalone-mixed-transaction/test.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ sentryTest(
3434
const traceId = transactionEnvelopeHeader.trace!.trace_id!;
3535
const parentSpanId = transactionEnvelopeItem.contexts?.trace?.span_id;
3636

37-
expect(traceId).toMatch(/[a-f0-9]{32}/);
38-
expect(parentSpanId).toMatch(/[a-f0-9]{16}/);
37+
expect(traceId).toMatch(/[a-f\d]{32}/);
38+
expect(parentSpanId).toMatch(/[a-f\d]{16}/);
3939

4040
expect(spanEnvelopeHeader).toEqual({
4141
sent_at: expect.any(String),
@@ -76,7 +76,7 @@ sentryTest(
7676
segment_id: transactionEnvelopeItem.contexts?.trace?.span_id,
7777
parent_span_id: parentSpanId,
7878
origin: 'manual',
79-
span_id: expect.stringMatching(/[a-f0-9]{16}/),
79+
span_id: expect.stringMatching(/[a-f\d]{16}/),
8080
start_timestamp: expect.any(Number),
8181
timestamp: expect.any(Number),
8282
trace_id: traceId,
@@ -111,7 +111,7 @@ sentryTest(
111111
description: 'inner',
112112
origin: 'manual',
113113
parent_span_id: parentSpanId,
114-
span_id: expect.stringMatching(/[a-f0-9]{16}/),
114+
span_id: expect.stringMatching(/[a-f\d]{16}/),
115115
start_timestamp: expect.any(Number),
116116
timestamp: expect.any(Number),
117117
trace_id: traceId,

‎dev-packages/browser-integration-tests/suites/public-api/startSpan/standalone/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ sentryTest('sends a segment span envelope', async ({ getLocalTestUrl, page }) =>
4848
},
4949
description: 'standalone_segment_span',
5050
origin: 'manual',
51-
span_id: expect.stringMatching(/^[0-9a-f]{16}$/),
51+
span_id: expect.stringMatching(/^[\da-f]{16}$/),
5252
start_timestamp: expect.any(Number),
5353
timestamp: expect.any(Number),
54-
trace_id: expect.stringMatching(/^[0-9a-f]{32}$/),
54+
trace_id: expect.stringMatching(/^[\da-f]{32}$/),
5555
is_segment: true,
5656
segment_id: spanJson.span_id,
5757
});

‎dev-packages/browser-integration-tests/suites/replay/dsc/test.ts‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ sentryTest(
5757
expect(envHeader.trace).toEqual({
5858
environment: 'production',
5959
sample_rate: '1',
60-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
60+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
6161
public_key: 'public',
6262
replay_id: replay.session?.id,
6363
sampled: 'true',
@@ -105,7 +105,7 @@ sentryTest(
105105
expect(envHeader.trace).toEqual({
106106
environment: 'production',
107107
sample_rate: '1',
108-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
108+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
109109
public_key: 'public',
110110
sampled: 'true',
111111
sample_rand: expect.any(String),
@@ -158,7 +158,7 @@ sentryTest(
158158
expect(envHeader.trace).toEqual({
159159
environment: 'production',
160160
sample_rate: '1',
161-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
161+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
162162
public_key: 'public',
163163
replay_id: replay.session?.id,
164164
sampled: 'true',
@@ -201,7 +201,7 @@ sentryTest(
201201
expect(envHeader.trace).toEqual({
202202
environment: 'production',
203203
sample_rate: '1',
204-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
204+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
205205
public_key: 'public',
206206
sampled: 'true',
207207
sample_rand: expect.any(String),
@@ -243,7 +243,7 @@ sentryTest('should add replay_id to error DSC while replay is active', async ({
243243
expect(error1Header.trace).toBeDefined();
244244
expect(error1Header.trace).toEqual({
245245
environment: 'production',
246-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
246+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
247247
public_key: 'public',
248248
replay_id: replay.session?.id,
249249
...(hasTracing
@@ -265,7 +265,7 @@ sentryTest('should add replay_id to error DSC while replay is active', async ({
265265
expect(error2Header.trace).toBeDefined();
266266
expect(error2Header.trace).toEqual({
267267
environment: 'production',
268-
trace_id: expect.stringMatching(/[a-f0-9]{32}/),
268+
trace_id: expect.stringMatching(/[a-f\d]{32}/),
269269
public_key: 'public',
270270
...(hasTracing
271271
? {

0 commit comments

Comments
 (0)