Merge pull request #8271 from getsentry/prepare-release/7.54.0

meta: Update changelog for 7.54.0

Author: Lms24

.github/workflows/build.yml

@@ -475,7 +475,7 @@ jobs:
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04
-    timeout-minutes: 18
+    timeout-minutes: 25
     strategy:
       fail-fast: false
       matrix:

.github/workflows/canary.yml

@@ -53,13 +53,14 @@ jobs:
           cd packages/e2e-tests
           yarn test:e2e
       - name: Create Issue
-        if: failure()
+        if: failure() && github.event_name == 'schedule'
         uses: JasonEtco/create-an-issue@e27dddc79c92bc6e4562f268fffa5ed752639abd
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           RUN_LINK: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
         with:
           filename: .github/CANARY_FAILURE_TEMPLATE.md
+          update_existing: true
 
   job_ember_canary_test:
     name: Ember Canary Tests
@@ -92,10 +93,12 @@ jobs:
           yarn ember try:one ${{ matrix.scenario }} --skip-cleanup=true
 
       - name: Create Issue
-        if: failure()
+        if: failure() && github.event_name == 'schedule'
         uses: JasonEtco/create-an-issue@e27dddc79c92bc6e4562f268fffa5ed752639abd
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           RUN_LINK: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
         with:
           filename: .github/CANARY_FAILURE_TEMPLATE.md
+          update_existing: true
+          title: 'Ember Canary tests failed'

CHANGELOG.md

@@ -4,6 +4,69 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 7.54.0
+
+### Important Changes
+
+- **feat(core): Add default entries to `ignoreTransactions` for Healthchecks #8191**
+
+  All SDKs now filter out health check transactions by default.
+  These are transactions where the transaction name matches typical API health check calls, such as `/^.*healthy.*$/` or `/^.  *heartbeat.*$/`. Take a look at [this list](https://github.com/getsentry/sentry-javascript/blob/8c6ad156829f7c4eec34e4a67e6dd866ba482d5d/packages/core/src/integrations/inboundfilters.ts#L8C2-L16) to learn which regexes we currently use to match transaction names.
+  We believe that these transactions do not provide value in most cases and we want to save you some of your quota by   filtering them out by default.
+  These filters are implemented as default values for the top level `ignoreTransactions` option.
+
+  You can disable this filtering by manually specifiying the `InboundFilters` integration and setting the   `disableTransactionDefaults` option:
+  ```js
+  Sentry.init({
+    //...
+    integrations: [new InboundFilters({ disableTransactionDefaults: true })],
+  })
+  ```
+
+- **feat(replay): Add `mutationBreadcrumbLimit` and `mutationLimit` to Replay Options (#8228)**
+
+  The previously experimental options `mutationBreadcumbLimit` and `mutationLimit` have been promoted to regular Replay   integration options.
+
+  A high number of DOM mutations (in a single event loop) can cause performance regressions in end-users' browsers.
+  Use `mutationBreadcrumbLimit` to send a breadcrumb along with your recording if the mutation limit was reached.
+  Use `mutationLimit` to stop recording if the mutation limit was reached.
+
+- **feat(sveltekit): Add source maps support for Vercel (lambda) (#8256)**
+  - feat(sveltekit): Auto-detect SvelteKit adapters (#8193)
+
+  The SvelteKit SDK can now be used if you deploy your SvelteKit app to Vercel.
+  By default, the SDK's Vite plugin will detect the used adapter and adjust the source map uploading config as necessary.
+  If you want to override the default adapter detection, you can specify the `adapter` option in the `sentrySvelteKit`  options:
+
+  ```js
+  // vite.config.js
+  export default defineConfig({
+    plugins: [
+      sentrySvelteKit({
+        adapter: 'vercel',
+      }),
+      sveltekit(),
+    ],
+  });
+  ```
+
+  Currently, the Vite plugin will configure itself correctly for `@sveltejs/adapter-auto`, `@sveltejs/adapter-vercel` and `@sveltejs/adapter-node`.
+
+  **Important:** The SvelteKit SDK is not yet compatible with Vercel's edge runtime.
+  It will only work for lambda functions.
+
+### Other Changes
+
+- feat(replay): Throttle breadcrumbs to max 300/5s (#8086)
+- feat(sveltekit): Add option to control handling of unknown server routes (#8201)
+- fix(node): Strip query and fragment from request URLs without route parameters (#8213)
+- fix(remix): Don't log missing parameters warning on server-side. (#8269)
+- fix(remix): Pass `loadContext` through wrapped document request function (#8268)
+- fix(replay): Guard against missing key (#8246)
+- fix(sveltekit): Avoid capturing redirects and 4xx Http errors in request Handlers (#8215)
+- fix(sveltekit): Bump `magicast` to support `satisfied` keyword (#8254)
+- fix(wasm): Avoid throwing an error when WASM modules are loaded from blobs (#8263)
+
 ## 7.53.1
 
 - chore(deps): bump socket.io-parser from 4.2.1 to 4.2.3 (#8196)

packages/browser-integration-tests/suites/replay/largeMutations/mutationLimit/init.js

@@ -4,9 +4,7 @@ window.Sentry = Sentry;
 window.Replay = new Sentry.Replay({
   flushMinDelay: 500,
   flushMaxDelay: 500,
-  _experiments: {
-    mutationLimit: 250,
-  },
+  mutationLimit: 250,
 });
 
 Sentry.init({

packages/browser-integration-tests/suites/replay/largeMutations/mutationLimit/test.ts

@@ -1,10 +1,15 @@
 import { expect } from '@playwright/test';
 
 import { sentryTest } from '../../../../utils/fixtures';
-import { getReplayRecordingContent, shouldSkipReplayTest, waitForReplayRequest } from '../../../../utils/replayHelpers';
+import {
+  getReplayRecordingContent,
+  getReplaySnapshot,
+  shouldSkipReplayTest,
+  waitForReplayRequest,
+} from '../../../../utils/replayHelpers';
 
 sentryTest(
-  'handles large mutations with _experiments.mutationLimit configured',
+  'handles large mutations by stopping replay when `mutationLimit` configured',
   async ({ getLocalTestPath, page, forceFlushReplay, browserName }) => {
     if (shouldSkipReplayTest() || ['webkit', 'firefox'].includes(browserName)) {
       sentryTest.skip();
@@ -34,36 +39,29 @@ sentryTest(
     await forceFlushReplay();
     const res1 = await reqPromise1;
 
-    const reqPromise2 = waitForReplayRequest(page);
+    // replay should be stopped due to mutation limit
+    let replay = await getReplaySnapshot(page);
+    expect(replay.session).toBe(undefined);
+    expect(replay._isEnabled).toBe(false);
 
     void page.click('#button-modify');
     await forceFlushReplay();
-    const res2 = await reqPromise2;
 
-    const reqPromise3 = waitForReplayRequest(page);
-
-    void page.click('#button-remove');
+    await page.click('#button-remove');
     await forceFlushReplay();
-    const res3 = await reqPromise3;
 
     const replayData0 = getReplayRecordingContent(res0);
-    const replayData1 = getReplayRecordingContent(res1);
-    const replayData2 = getReplayRecordingContent(res2);
-    const replayData3 = getReplayRecordingContent(res3);
-
     expect(replayData0.fullSnapshots.length).toBe(1);
     expect(replayData0.incrementalSnapshots.length).toBe(0);
 
-    // This includes both a full snapshot as well as some incremental snapshots
-    expect(replayData1.fullSnapshots.length).toBe(1);
+    // Breadcrumbs (click and mutation);
+    const replayData1 = getReplayRecordingContent(res1);
+    expect(replayData1.fullSnapshots.length).toBe(0);
     expect(replayData1.incrementalSnapshots.length).toBeGreaterThan(0);
+    expect(replayData1.breadcrumbs.map(({ category }) => category).sort()).toEqual(['replay.mutations', 'ui.click']);
 
-    // This does not trigger mutations, for whatever reason - so no full snapshot either!
-    expect(replayData2.fullSnapshots.length).toBe(0);
-    expect(replayData2.incrementalSnapshots.length).toBeGreaterThan(0);
-
-    // This includes both a full snapshot as well as some incremental snapshots
-    expect(replayData3.fullSnapshots.length).toBe(1);
-    expect(replayData3.incrementalSnapshots.length).toBeGreaterThan(0);
+    replay = await getReplaySnapshot(page);
+    expect(replay.session).toBe(undefined);
+    expect(replay._isEnabled).toBe(false);
   },
 );

packages/browser-integration-tests/suites/replay/sessionExpiry/test.ts

@@ -14,8 +14,9 @@ import {
 // Session should expire after 2s - keep in sync with init.js
 const SESSION_TIMEOUT = 2000;
 
-sentryTest('handles an expired session', async ({ getLocalTestPath, page }) => {
-  if (shouldSkipReplayTest()) {
+sentryTest('handles an expired session', async ({ browserName, getLocalTestPath, page }) => {
+  // This test seems to only be flakey on firefox
+  if (shouldSkipReplayTest() || ['firefox'].includes(browserName)) {
     sentryTest.skip();
   }
 

packages/browser-integration-tests/suites/replay/slowClick/mutation/test.ts

@@ -63,8 +63,9 @@ sentryTest('mutation after threshold results in slow click', async ({ getLocalTe
   expect(slowClickBreadcrumbs[0]?.data?.timeAfterClickMs).toBeLessThan(2000);
 });
 
-sentryTest('immediate mutation does not trigger slow click', async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipReplayTest()) {
+sentryTest('immediate mutation does not trigger slow click', async ({ browserName, getLocalTestUrl, page }) => {
+  // This test seems to only be flakey on firefox
+  if (shouldSkipReplayTest() || ['firefox'].includes(browserName)) {
     sentryTest.skip();
   }
 

packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/init.js

@@ -0,0 +1,17 @@
+import * as Sentry from '@sentry/browser';
+
+window.Sentry = Sentry;
+window.Replay = new Sentry.Replay({
+  flushMinDelay: 5000,
+  flushMaxDelay: 5000,
+  useCompression: false,
+});
+
+Sentry.init({
+  dsn: 'https://[email protected]/1337',
+  sampleRate: 0,
+  replaysSessionSampleRate: 1.0,
+  replaysOnErrorSampleRate: 0.0,
+
+  integrations: [window.Replay],
+});

packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/subject.js

@@ -0,0 +1,8 @@
+const COUNT = 400;
+
+document.querySelector('[data-console]').addEventListener('click', () => {
+  // Call console.log() many times
+  for (let i = 0; i < COUNT; i++) {
+    console.log(`testing ${i}`);
+  }
+});

packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/template.html

@@ -0,0 +1,9 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+  </head>
+  <body>
+    <button data-console>Trigger console.log</button>
+  </body>
+</html>