reverting to commit 22b5cbf

Author: c298lee

CHANGELOG.md

@@ -4,55 +4,6 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
-## 7.72.0
-
-### Important Changes
-
-- **feat(node): App Not Responding with stack traces (#9079)**
-
-This release introduces support for Application Not Responding (ANR) errors for Node.js applications.
-These errors are triggered when the Node.js main thread event loop of an application is blocked for more than five seconds.
-The Node SDK reports ANR errors as Sentry events and can optionally attach a stacktrace of the blocking code to the ANR event.
-
-To enable ANR detection, import and use the `enableANRDetection` function from the `@sentry/node` package before you run the rest of your application code.
-Any event loop blocking before calling `enableANRDetection` will not be detected by the SDK.
-
-Example (ESM):
-
-```ts
-import * as Sentry from "@sentry/node";
-
-Sentry.init({
-  dsn: "___PUBLIC_DSN___",
-  tracesSampleRate: 1.0,
-});
-
-await Sentry.enableANRDetection({ captureStackTrace: true });
-// Function that runs your app
-runApp();
-```
-
-Example (CJS):
-
-```ts
-const Sentry = require("@sentry/node");
-
-Sentry.init({
-  dsn: "___PUBLIC_DSN___",
-  tracesSampleRate: 1.0,
-});
-
-Sentry.enableANRDetection({ captureStackTrace: true }).then(() => {
-  // Function that runs your app
-  runApp();
-});
-```
-
-### Other Changes
-
-- fix(nextjs): Filter `RequestAsyncStorage` locations by locations that webpack will resolve (#9114)
-- fix(replay): Ensure `replay_id` is not captured when session is expired (#9109)
-
 ## 7.71.0
 
 - feat(bun): Instrument Bun.serve (#9080)

lerna.json

@@ -1,5 +1,5 @@
 {
   "$schema": "node_modules/lerna/schemas/lerna-schema.json",
-  "version": "7.72.0",
+  "version": "7.71.0",
   "npmClient": "yarn"
 }

packages/angular-ivy/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/angular-ivy",
-  "version": "7.72.0",
+  "version": "7.71.0",
   "description": "Official Sentry SDK for Angular with full Ivy Support",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular-ivy",
@@ -21,9 +21,9 @@
     "rxjs": "^6.5.5 || ^7.x"
   },
   "dependencies": {
-    "@sentry/browser": "7.72.0",
-    "@sentry/types": "7.72.0",
-    "@sentry/utils": "7.72.0",
+    "@sentry/browser": "7.71.0",
+    "@sentry/types": "7.71.0",
+    "@sentry/utils": "7.71.0",
     "tslib": "^2.4.1"
   },
   "devDependencies": {

packages/angular/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/angular",
-  "version": "7.72.0",
+  "version": "7.71.0",
   "description": "Official Sentry SDK for Angular",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular",
@@ -21,9 +21,9 @@
     "rxjs": "^6.5.5 || ^7.x"
   },
   "dependencies": {
-    "@sentry/browser": "7.72.0",
-    "@sentry/types": "7.72.0",
-    "@sentry/utils": "7.72.0",
+    "@sentry/browser": "7.71.0",
+    "@sentry/types": "7.71.0",
+    "@sentry/utils": "7.71.0",
     "tslib": "^2.4.1"
   },
   "devDependencies": {

packages/browser-integration-tests/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry-internal/browser-integration-tests",
-  "version": "7.72.0",
+  "version": "7.71.0",
   "main": "index.js",
   "license": "MIT",
   "engines": {

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

@@ -84,13 +84,12 @@ sentryTest(
     await reqErrorPromise;
     expect(callsToSentry).toEqual(2);
 
-    const [req0] = await Promise.all([
-      reqPromise0,
-      page.evaluate(async () => {
-        const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
-        await replayIntegration.flush();
-      }),
-    ]);
+    await page.evaluate(async () => {
+      const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
+      await replayIntegration.flush();
+    });
+
+    const req0 = await reqPromise0;
 
     // 2 errors, 1 flush
     await reqErrorPromise;
@@ -227,13 +226,12 @@ sentryTest(
     await reqErrorPromise;
     expect(callsToSentry).toEqual(2);
 
-    const [req0] = await Promise.all([
-      reqPromise0,
-      page.evaluate(async () => {
-        const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
-        await replayIntegration.flush({ continueRecording: false });
-      }),
-    ]);
+    await page.evaluate(async () => {
+      const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
+      await replayIntegration.flush({ continueRecording: false });
+    });
+
+    const req0 = await reqPromise0;
 
     // 2 errors, 1 flush
     await reqErrorPromise;
@@ -348,12 +346,9 @@ sentryTest(
 
     // Error sample rate is now at 1.0, this error should create a replay
     const reqErrorPromise1 = waitForErrorRequest(page);
-    const [req0] = await Promise.all([
-      // 1 unsampled error, 1 sampled error -> 1 flush
-      reqPromise0,
-      page.click('#error2'),
-    ]);
-
+    await page.click('#error2');
+    // 1 unsampled error, 1 sampled error -> 1 flush
+    const req0 = await reqPromise0;
     const reqError1 = await reqErrorPromise1;
     const errorEvent1 = envelopeRequestParser(reqError1);
     expect(callsToSentry).toEqual(3);

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

@@ -10,6 +10,8 @@ sentryTest('should capture console messages in replay', async ({ getLocalTestPat
     sentryTest.skip();
   }
 
+  const reqPromise0 = waitForReplayRequest(page, 0);
+
   await page.route('https://dsn.ingest.sentry.io/**/*', route => {
     return route.fulfill({
       status: 200,
@@ -18,11 +20,10 @@ sentryTest('should capture console messages in replay', async ({ getLocalTestPat
     });
   });
 
-  const reqPromise0 = waitForReplayRequest(page, 0);
-
   const url = await getLocalTestPath({ testDir: __dirname });
 
-  await Promise.all([page.goto(url), reqPromise0]);
+  await page.goto(url);
+  await reqPromise0;
 
   const reqPromise1 = waitForReplayRequest(
     page,
@@ -37,10 +38,11 @@ sentryTest('should capture console messages in replay', async ({ getLocalTestPat
   await page.click('[data-log]');
 
   // Sometimes this doesn't seem to trigger, so we trigger it twice to be sure...
-  const [req1] = await Promise.all([reqPromise1, page.click('[data-log]')]);
+  await page.click('[data-log]');
+
   await forceFlushReplay();
 
-  const { breadcrumbs } = getCustomRecordingEvents(req1);
+  const { breadcrumbs } = getCustomRecordingEvents(await reqPromise1);
 
   expect(breadcrumbs.filter(breadcrumb => breadcrumb.category === 'console')).toEqual(
     expect.arrayContaining([
@@ -63,6 +65,8 @@ sentryTest('should capture very large console logs', async ({ getLocalTestPath,
     sentryTest.skip();
   }
 
+  const reqPromise0 = waitForReplayRequest(page, 0);
+
   await page.route('https://dsn.ingest.sentry.io/**/*', route => {
     return route.fulfill({
       status: 200,
@@ -71,11 +75,10 @@ sentryTest('should capture very large console logs', async ({ getLocalTestPath,
     });
   });
 
-  const reqPromise0 = waitForReplayRequest(page, 0);
-
   const url = await getLocalTestPath({ testDir: __dirname });
 
-  await Promise.all([page.goto(url), reqPromise0]);
+  await page.goto(url);
+  await reqPromise0;
 
   const reqPromise1 = waitForReplayRequest(
     page,
@@ -87,10 +90,14 @@ sentryTest('should capture very large console logs', async ({ getLocalTestPath,
     5_000,
   );
 
-  const [req1] = await Promise.all([reqPromise1, page.click('[data-log-large]')]);
+  await page.click('[data-log-large]');
+
+  // Sometimes this doesn't seem to trigger, so we trigger it twice to be sure...
+  await page.click('[data-log-large]');
+
   await forceFlushReplay();
 
-  const { breadcrumbs } = getCustomRecordingEvents(req1);
+  const { breadcrumbs } = getCustomRecordingEvents(await reqPromise1);
 
   expect(breadcrumbs.filter(breadcrumb => breadcrumb.category === 'console')).toEqual(
     expect.arrayContaining([

packages/browser-integration-tests/suites/replay/errors/errorMode/test.ts

@@ -50,25 +50,26 @@ sentryTest(
 
     const url = await getLocalTestPath({ testDir: __dirname });
 
-    await Promise.all([
-      page.goto(url),
-      page.click('#go-background'),
-      new Promise(resolve => setTimeout(resolve, 1000)),
-    ]);
+    await page.goto(url);
+    await page.click('#go-background');
+    await new Promise(resolve => setTimeout(resolve, 1000));
 
     expect(callsToSentry).toEqual(0);
 
-    const [req0] = await Promise.all([reqPromise0, page.click('#error')]);
+    await page.click('#error');
+    const req0 = await reqPromise0;
 
     expect(callsToSentry).toEqual(2); // 1 error, 1 replay event
 
-    const [req1] = await Promise.all([reqPromise1, page.click('#go-background'), reqErrorPromise]);
+    await page.click('#go-background');
+    const req1 = await reqPromise1;
+    await reqErrorPromise;
 
     expect(callsToSentry).toEqual(3); // 1 error, 2 replay events
 
     await page.click('#log');
-
-    const [req2] = await Promise.all([reqPromise2, page.click('#go-background')]);
+    await page.click('#go-background');
+    const req2 = await reqPromise2;
 
     const event0 = getReplayEvent(req0);
     const content0 = getReplayRecordingContent(req0);

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

@@ -10,6 +10,8 @@ sentryTest(
       sentryTest.skip();
     }
 
+    const reqPromise0 = waitForReplayRequest(page, 0);
+
     await page.route('https://dsn.ingest.sentry.io/**/*', route => {
       return route.fulfill({
         status: 200,
@@ -20,17 +22,26 @@ sentryTest(
 
     const url = await getLocalTestPath({ testDir: __dirname });
 
-    const [res0] = await Promise.all([waitForReplayRequest(page, 0), page.goto(url)]);
-    await forceFlushReplay();
+    await page.goto(url);
+    const res0 = await reqPromise0;
+
+    const reqPromise1 = waitForReplayRequest(page);
 
-    const [res1] = await Promise.all([waitForReplayRequest(page), page.click('#button-add')]);
+    void page.click('#button-add');
     await forceFlushReplay();
+    const res1 = await reqPromise1;
 
-    const [res2] = await Promise.all([waitForReplayRequest(page), page.click('#button-modify')]);
+    const reqPromise2 = waitForReplayRequest(page);
+
+    void page.click('#button-modify');
     await forceFlushReplay();
+    const res2 = await reqPromise2;
+
+    const reqPromise3 = waitForReplayRequest(page);
 
-    const [res3] = await Promise.all([waitForReplayRequest(page), page.click('#button-remove')]);
+    void page.click('#button-remove');
     await forceFlushReplay();
+    const res3 = await reqPromise3;
 
     const replayData0 = getReplayRecordingContent(res0);
     const replayData1 = getReplayRecordingContent(res1);

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

@@ -15,6 +15,8 @@ sentryTest(
       sentryTest.skip();
     }
 
+    const reqPromise0 = waitForReplayRequest(page, 0);
+
     await page.route('https://dsn.ingest.sentry.io/**/*', route => {
       return route.fulfill({
         status: 200,
@@ -23,24 +25,23 @@ sentryTest(
       });
     });
 
-    const reqPromise0 = waitForReplayRequest(page, 0);
-
     const url = await getLocalTestPath({ testDir: __dirname });
 
-    const [res0] = await Promise.all([reqPromise0, page.goto(url)]);
-    await forceFlushReplay();
+    await page.goto(url);
+    const res0 = await reqPromise0;
 
     const reqPromise1 = waitForReplayRequest(page);
 
-    const [res1] = await Promise.all([reqPromise1, page.click('#button-add')]);
+    void page.click('#button-add');
     await forceFlushReplay();
+    const res1 = await reqPromise1;
 
     // replay should be stopped due to mutation limit
     let replay = await getReplaySnapshot(page);
     expect(replay.session).toBe(undefined);
     expect(replay._isEnabled).toBe(false);
 
-    await page.click('#button-modify');
+    void page.click('#button-modify');
     await forceFlushReplay();
 
     await page.click('#button-remove');