Merge branch 'develop' into onur/fix-flaky-remix-e2e-tests

Author: AbhiPrasad

MIGRATION.md

@@ -1,5 +1,21 @@
 # Upgrading from 7.x to 8.x
 
+## Removal of deprecated API in `@sentry/nextjs`
+
+The following previously deprecated API has been removed from the `@sentry/nextjs` package:
+
+- `withSentryApi` (Replacement: `wrapApiHandlerWithSentry`)
+- `withSentryAPI` (Replacement: `wrapApiHandlerWithSentry`)
+- `withSentryGetServerSideProps` (Replacement: `wrapGetServerSidePropsWithSentry`)
+- `withSentryGetStaticProps` (Replacement: `wrapGetStaticPropsWithSentry`)
+- `withSentryServerSideGetInitialProps` (Replacement: `wrapGetInitialPropsWithSentry`)
+- `withSentryServerSideAppGetInitialProps` (Replacement: `wrapAppGetInitialPropsWithSentry`)
+- `withSentryServerSideDocumentGetInitialProps` (Replacement: `wrapDocumentGetInitialPropsWithSentry`)
+- `withSentryServerSideErrorGetInitialProps` was renamed to `wrapErrorGetInitialPropsWithSentry`
+- `nextRouterInstrumentation` (Replaced by using `browserTracingIntegration`)
+- `IS_BUILD`
+- `isBuild`
+
 ## Removal of Severity Enum
 
 In v7 we deprecated the `Severity` enum in favor of using the `SeverityLevel` type. In v8 we removed the `Severity`
@@ -11,6 +27,12 @@ enum. If you were using the `Severity` enum, you should replace it with the `Sev
 The `Offline` integration has been removed in favor of the offline transport wrapper:
 http://docs.sentry.io/platforms/javascript/configuration/transports/#offline-caching
 
+## Removal of `enableAnrDetection` and `Anr` class (##10562)
+
+The `enableAnrDetection` and `Anr` class have been removed. See the
+[docs](https://docs.sentry.io/platforms/node/configuration/application-not-responding/) for more details how to migrate
+to `anrIntegration`, the new integration for ANR detection.
+
 ## Other changes
 
 - Remove `spanStatusfromHttpCode` in favour of `getSpanStatusFromHttpCode` (#10361)
@@ -289,6 +311,45 @@ If you are using the `Hub` right now, see the following table on how to migrate
 | endSession()           | `Sentry.endSession()`                                                                |
 | shouldSendDefaultPii() | REMOVED - The closest equivalent is `Sentry.getClient().getOptions().sendDefaultPii` |
 
+The `Hub` constructor is also deprecated and will be removed in the next major version. If you are creating Hubs for
+multi-client use like so:
+
+```ts
+// OLD
+const hub = new Hub();
+hub.bindClient(client);
+makeMain(hub);
+```
+
+instead initialize the client as follows:
+
+```ts
+// NEW
+Sentry.withIsolationScope(() => {
+  Sentry.setCurrentClient(client);
+  client.init();
+});
+```
+
+If you are using the Hub to capture events like so:
+
+```ts
+// OLD
+const client = new Client();
+const hub = new Hub(client);
+hub.captureException();
+```
+
+instead capture isolated events as follows:
+
+```ts
+// NEW
+const client = new Client();
+const scope = new Scope();
+scope.setClient(client);
+scope.captureException();
+```
+
 ## Deprecate `client.setupIntegrations()`
 
 Instead, use the new `client.init()` method. You should probably not use this directly and instead use `Sentry.init()`,

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

@@ -36,7 +36,7 @@ sentryTest(
 
     await page.evaluate(() => {
       const scope = (window as unknown as TestWindow).Sentry.getCurrentScope();
-      scope.setUser({ id: 'user123', segment: 'segmentB' });
+      scope.setUser({ id: 'user123' });
       scope.addEventProcessor(event => {
         event.transaction = 'testTransactionDSC';
         return event;
@@ -53,7 +53,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       trace_id: expect.any(String),
       public_key: 'public',
@@ -84,7 +83,7 @@ sentryTest(
 
     await page.evaluate(() => {
       const scope = (window as unknown as TestWindow).Sentry.getCurrentScope();
-      scope.setUser({ id: 'user123', segment: 'segmentB' });
+      scope.setUser({ id: 'user123' });
       scope.addEventProcessor(event => {
         event.transaction = 'testTransactionDSC';
         return event;
@@ -101,7 +100,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       trace_id: expect.any(String),
       public_key: 'public',
@@ -144,7 +142,7 @@ sentryTest(
 
     await page.evaluate(() => {
       const scope = (window as unknown as TestWindow).Sentry.getCurrentScope();
-      scope.setUser({ id: 'user123', segment: 'segmentB' });
+      scope.setUser({ id: 'user123' });
       scope.addEventProcessor(event => {
         event.transaction = 'testTransactionDSC';
         return event;
@@ -162,7 +160,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       trace_id: expect.any(String),
       public_key: 'public',
@@ -195,7 +192,7 @@ sentryTest(
 
     await page.evaluate(async () => {
       const scope = (window as unknown as TestWindow).Sentry.getCurrentScope();
-      scope.setUser({ id: 'user123', segment: 'segmentB' });
+      scope.setUser({ id: 'user123' });
       scope.addEventProcessor(event => {
         event.transaction = 'testTransactionDSC';
         return event;
@@ -213,7 +210,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       trace_id: expect.any(String),
       public_key: 'public',

dev-packages/browser-integration-tests/suites/tracing/envelope-header-transaction-name/init.js

@@ -12,7 +12,7 @@ Sentry.init({
 });
 
 const scope = Sentry.getCurrentScope();
-scope.setUser({ id: 'user123', segment: 'segmentB' });
+scope.setUser({ id: 'user123' });
 scope.addEventProcessor(event => {
   event.transaction = 'testTransactionDSC';
   return event;

dev-packages/browser-integration-tests/suites/tracing/envelope-header-transaction-name/test.ts

@@ -22,7 +22,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       transaction: expect.stringContaining('/index.html'),
       trace_id: expect.any(String),

dev-packages/browser-integration-tests/suites/tracing/envelope-header/init.js

@@ -12,7 +12,7 @@ Sentry.init({
 });
 
 const scope = Sentry.getCurrentScope();
-scope.setUser({ id: 'user123', segment: 'segmentB' });
+scope.setUser({ id: 'user123' });
 scope.addEventProcessor(event => {
   event.transaction = 'testTransactionDSC';
   return event;

dev-packages/browser-integration-tests/suites/tracing/envelope-header/test.ts

@@ -26,7 +26,6 @@ sentryTest(
     expect(envHeader.trace).toBeDefined();
     expect(envHeader.trace).toEqual({
       environment: 'production',
-      user_segment: 'segmentB',
       sample_rate: '1',
       trace_id: expect.any(String),
       public_key: 'public',

dev-packages/e2e-tests/test-applications/create-next-app/pages/api/success.ts

@@ -3,16 +3,7 @@ import * as Sentry from '@sentry/nextjs';
 import type { NextApiRequest, NextApiResponse } from 'next';
 
 export default function handler(req: NextApiRequest, res: NextApiResponse) {
-  // eslint-disable-next-line deprecation/deprecation
-  const transaction = Sentry.startTransaction({ name: 'test-transaction', op: 'e2e-test' });
-  // eslint-disable-next-line deprecation/deprecation
-  Sentry.getCurrentScope().setSpan(transaction);
-
-  // eslint-disable-next-line deprecation/deprecation
-  const span = transaction.startChild();
-
-  span.end();
-  transaction.end();
+  Sentry.startSpan({ name: 'test-span' }, span => undefined);
 
   Sentry.flush().then(() => {
     res.status(200).json({

dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts

@@ -60,7 +60,7 @@ const DEPENDENTS: Dependent[] = [
   {
     package: '@sentry/serverless',
     exports: Object.keys(SentryServerless),
-    ignoreExports: ['cron', 'hapiErrorPlugin', 'enableAnrDetection'],
+    ignoreExports: ['cron', 'hapiErrorPlugin'],
   },
   {
     package: '@sentry/sveltekit',

dev-packages/node-integration-tests/suites/anr/basic-session.js

@@ -11,7 +11,7 @@ Sentry.init({
   dsn: 'https://[email protected]/1337',
   release: '1.0',
   debug: true,
-  integrations: [new Sentry.Integrations.Anr({ captureStackTrace: true, anrThreshold: 100 })],
+  integrations: [Sentry.anrIntegration({ captureStackTrace: true, anrThreshold: 100 })],
 });
 
 function longWork() {

dev-packages/node-integration-tests/suites/anr/basic.js

@@ -12,7 +12,7 @@ Sentry.init({
   release: '1.0',
   debug: true,
   autoSessionTracking: false,
-  integrations: [new Sentry.Integrations.Anr({ captureStackTrace: true, anrThreshold: 100 })],
+  integrations: [Sentry.anrIntegration({ captureStackTrace: true, anrThreshold: 100 })],
 });
 
 function longWork() {