Skip to content

Commit 8a06050

Browse files
authored
ref(v9): Remove v9 todos (#12740)
* ref(v9): Remove v9 todos * Undo electron changes in favor of #12655 * Explain how to disable sending sessions in the node SDK
1 parent bbab4dd commit 8a06050

File tree

2 files changed

+5
-23
lines changed

2 files changed

+5
-23
lines changed

platform-includes/configuration/auto-session-tracking/javascript.node.mdx

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -7,24 +7,14 @@ We mark the session as crashed if an _unhandled error_ reached our `errorHandler
77

88
We mark the session as an error if the SDK captures an event that contains an exception (this includes manually captured exceptions).
99

10-
By default, the JavaScript SDKs are sending sessions, to disable this toggle the flag `autoSessionTracking` to `false`:
10+
By default, the SDK is sending sessions, to disable this configure the `httpIntegration` with the `trackIncomingRequestsAsSessions` option set to `false`.
1111

1212
```javascript
1313
Sentry.init({
14-
autoSessionTracking: false, // default: true
14+
integrations: [Sentry.httpIntegration({
15+
trackIncomingRequestsAsSessions: false, // default: true
16+
})]
1517
});
1618
```
1719

18-
{/* TODO(v9): Remove this notice? */}
19-
20-
<PlatformSection notSupported={["javacript.cordova", "javascript.capacitor", "javascript.electron"]}>
21-
22-
<Alert title="Deprecation Notice">
23-
24-
{/* We don't use platform links here because we always have to link to node and browser here and doing a conditional just for that feels overkill. */}
25-
26-
Starting with SDK version 8.43.0 and up, the `autoSessionTracking` option has been deprecated. You can use the [BrowserSession integration](/platforms/javascript/configuration/integrations/browsersession/) in browser environments and the [Http integration](/platforms/javascript/guides/node/configuration/integrations/http/) (via the `trackIncomingRequestsAsSessions` option) in Node.js runtime.
27-
28-
</Alert>
29-
30-
</PlatformSection>
20+
A session is automatically created for every node process by the `processSessionIntegration` which is configured by default.

platform-includes/getting-started-use/javascript.nestjs.mdx

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,6 @@ export class YourCatchAllExceptionFilter implements ExceptionFilter {
5252
By default, only unhandled exceptions that are not caught by an error filter are reported to Sentry.
5353
`HttpException`s (including [derivatives](https://docs.nestjs.com/exception-filters#built-in-http-exceptions)) are also not captured by default because they mostly act as control flow vehicles.
5454

55-
{/* TODO(v9): Remove this note */}
56-
_Note that `@SentryExceptionCaptured()` was called `@WithSentry` in SDK versions `8.38.0` and prior._
57-
5855
If you don't have a global catch-all exception filter, add the `SentryGlobalFilter` to the providers of your main module.
5956
This filter will report any unhandled errors that aren't caught by other error filters to Sentry.
6057
**Important:** The `SentryGlobalFilter` needs to be registered before any other exception filters.
@@ -76,11 +73,6 @@ import { SentryGlobalFilter } from "@sentry/nestjs/setup";
7673
export class AppModule {}
7774
```
7875

79-
{/* TODO(v9): Remove this note. */}
80-
81-
**Note:** If you have a NestJS + GraphQL application and you are using the `@sentry/nestjs` SDK version `8.38.0` or earlier, replace the `SentryGlobalFilter` with the `SentryGlobalGenericFilter`.
82-
In SDK versions `8.39.0` and above, the `SentryGlobalGenericFilter` is deprecated because the `SentryGlobalFilter` will handle GraphQL contexts automatically.
83-
8476
If you have error filters for specific types of exceptions (for example `@Catch(HttpException)`, or any other `@Catch(...)` with arguments) and you want to capture errors caught by these filters, capture the errors in the `catch()` handler with `Sentry.captureException()`:
8577

8678
```javascript {9}

0 commit comments

Comments
 (0)