Skip to content

Commit 5ec50f6

Browse files
authored
Merge branch 'develop' into fix/fetch-not-release
2 parents 8c99aee + 6410841 commit 5ec50f6

File tree

79 files changed

+716
-3733
lines changed

Some content is hidden

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

79 files changed

+716
-3733
lines changed

.github/dependabot.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,10 @@ updates:
1313
schedule:
1414
interval: 'weekly'
1515
allow:
16-
- dependency-name: "@sentry/cli"
17-
- dependency-name: "@sentry/vite-plugin"
18-
- dependency-name: "@sentry/webpack-plugin"
19-
- dependency-name: "@sentry/rollup-plugin"
20-
- dependency-name: "@sentry/esbuild-plugin"
16+
- dependency-name: "@sentry/*"
2117
- dependency-name: "@opentelemetry/*"
2218
- dependency-name: "@prisma/instrumentation"
19+
- dependency-name: "opentelemetry-instrumentation-remix"
2320
versioning-strategy: increase
2421
commit-message:
2522
prefix: feat

.github/workflows/build.yml

Lines changed: 4 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,7 @@ jobs:
437437
with:
438438
node-version-file: 'package.json'
439439
- name: Set up Deno
440-
uses: denoland/setup-deno@v1.5.1
440+
uses: denoland/setup-deno@v2.0.1
441441
with:
442442
deno-version: v1.38.5
443443
- name: Restore caches
@@ -858,7 +858,7 @@ jobs:
858858
if: always() && needs.job_e2e_prepare.result == 'success'
859859
needs: [job_get_metadata, job_build, job_e2e_prepare]
860860
runs-on: ubuntu-20.04
861-
timeout-minutes: 10
861+
timeout-minutes: 15
862862
env:
863863
# We just use a dummy DSN here, only send to the tunnel anyhow
864864
E2E_TEST_DSN: 'https://username@domain/123'
@@ -1070,7 +1070,7 @@ jobs:
10701070
github.actor != 'dependabot[bot]'
10711071
needs: [job_get_metadata, job_build, job_e2e_prepare]
10721072
runs-on: ubuntu-20.04
1073-
timeout-minutes: 10
1073+
timeout-minutes: 15
10741074
env:
10751075
E2E_TEST_AUTH_TOKEN: ${{ secrets.E2E_TEST_AUTH_TOKEN }}
10761076
E2E_TEST_DSN: ${{ secrets.E2E_TEST_DSN }}
@@ -1234,7 +1234,7 @@ jobs:
12341234
)
12351235
needs: [job_get_metadata, job_build, job_e2e_prepare]
12361236
runs-on: ubuntu-20.04
1237-
timeout-minutes: 10
1237+
timeout-minutes: 15
12381238
env:
12391239
E2E_TEST_AUTH_TOKEN: ${{ secrets.E2E_TEST_AUTH_TOKEN }}
12401240
E2E_TEST_DSN: ${{ secrets.E2E_TEST_DSN }}
@@ -1345,48 +1345,6 @@ jobs:
13451345
run: |
13461346
echo "One of the dependent jobs have failed. You may need to re-run it." && exit 1
13471347
1348-
overhead_metrics:
1349-
name: Overhead metrics
1350-
needs: [job_get_metadata, job_build]
1351-
runs-on: ubuntu-20.04
1352-
timeout-minutes: 30
1353-
if: |
1354-
contains(github.event.pull_request.labels.*.name, 'ci-overhead-measurements')
1355-
steps:
1356-
- name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
1357-
uses: actions/checkout@v4
1358-
with:
1359-
ref: ${{ env.HEAD_COMMIT }}
1360-
- name: Set up Node
1361-
uses: actions/setup-node@v4
1362-
with:
1363-
node-version-file: 'package.json'
1364-
- name: Restore caches
1365-
uses: ./.github/actions/restore-cache
1366-
with:
1367-
dependency_cache_key: ${{ needs.job_build.outputs.dependency_cache_key }}
1368-
1369-
- name: Collect
1370-
run: yarn ci:collect
1371-
working-directory: dev-packages/overhead-metrics
1372-
1373-
- name: Process
1374-
id: process
1375-
run: yarn ci:process
1376-
working-directory: dev-packages/overhead-metrics
1377-
# Don't run on forks - the PR comment cannot be added.
1378-
if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
1379-
env:
1380-
GITHUB_TOKEN: ${{ github.token }}
1381-
1382-
- name: Upload results
1383-
uses: actions/upload-artifact@v4
1384-
if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
1385-
with:
1386-
name: ${{ steps.process.outputs.artifactName }}
1387-
path: ${{ steps.process.outputs.artifactPath }}
1388-
retention-days: 7
1389-
13901348
job_compile_bindings_profiling_node:
13911349
name: Compile & Test Profiling Bindings (v${{ matrix.node }}) ${{ matrix.target_platform || matrix.os }}, ${{ matrix.node || matrix.container }}, ${{ matrix.arch || matrix.container }}, ${{ contains(matrix.container, 'alpine') && 'musl' || 'glibc' }}
13921350
needs: [job_get_metadata, job_build]
Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import * as Sentry from '@sentry/browser';
2+
3+
window.Sentry = Sentry;
4+
5+
window.fetchCallCount = 0;
6+
window.spanEnded = false;
7+
8+
const originalWindowFetch = window.fetch;
9+
window.fetch = (...args) => {
10+
window.fetchCallCount++;
11+
return originalWindowFetch(...args);
12+
};
13+
14+
Sentry.init({
15+
dsn: 'https://[email protected]/1337',
16+
tracesSampleRate: 1.0,
17+
enabled: false,
18+
});
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
Sentry.startSpan({ name: 'standalone_segment_span', experimental: { standalone: true } }, () => {});
2+
3+
window.spanEnded = true;
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
import { expect } from '@playwright/test';
2+
3+
import { sentryTest } from '../../../../utils/fixtures';
4+
import { shouldSkipTracingTest } from '../../../../utils/helpers';
5+
6+
sentryTest("doesn't send a standalone span envelope if SDK is disabled", async ({ getLocalTestPath, page }) => {
7+
if (shouldSkipTracingTest()) {
8+
sentryTest.skip();
9+
}
10+
11+
const url = await getLocalTestPath({ testDir: __dirname });
12+
await page.goto(url);
13+
14+
// @ts-expect-error this exists in the test init/subject
15+
await page.waitForFunction(() => !!window.spanEnded);
16+
await page.waitForTimeout(2000);
17+
18+
// @ts-expect-error this exists in the test init
19+
const fetchCallCount = await page.evaluate(() => window.fetchCallCount);
20+
// We expect no fetch calls because the SDK is disabled
21+
expect(fetchCallCount).toBe(0);
22+
});

dev-packages/e2e-tests/test-applications/debug-id-sourcemaps/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"@sentry/node": "latest || *"
1414
},
1515
"devDependencies": {
16-
"rollup": "^4.0.2",
16+
"rollup": "^4.24.2",
1717
"vitest": "^0.34.6",
1818
"@sentry/rollup-plugin": "2.22.6"
1919
},

dev-packages/e2e-tests/test-applications/nextjs-13/sentry.client.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client';
2+
13
import * as Sentry from '@sentry/nextjs';
24

35
Sentry.init({

dev-packages/e2e-tests/test-applications/nextjs-14/sentry.client.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client';
2+
13
import * as Sentry from '@sentry/nextjs';
24

35
Sentry.init({

dev-packages/e2e-tests/test-applications/nextjs-15/sentry.client.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client';
2+
13
import * as Sentry from '@sentry/nextjs';
24

35
Sentry.init({

dev-packages/e2e-tests/test-applications/nextjs-app-dir/sentry.client.config.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use client';
2+
13
import * as Sentry from '@sentry/nextjs';
24

35
Sentry.init({

0 commit comments

Comments
 (0)