Remove SentryGlobalGraphQLFilter

Author: andreiborza

dev-packages/e2e-tests/test-applications/nestjs-graphql/src/app.module.ts

@@ -2,7 +2,7 @@ import { ApolloDriver } from '@nestjs/apollo';
 import { Logger, Module } from '@nestjs/common';
 import { APP_FILTER } from '@nestjs/core';
 import { GraphQLModule } from '@nestjs/graphql';
-import { SentryGlobalGraphQLFilter, SentryModule } from '@sentry/nestjs/setup';
+import { SentryGlobalFilter, SentryModule } from '@sentry/nestjs/setup';
 import { AppResolver } from './app.resolver';
 
 @Module({
@@ -19,7 +19,7 @@ import { AppResolver } from './app.resolver';
     AppResolver,
     {
       provide: APP_FILTER,
-      useClass: SentryGlobalGraphQLFilter,
+      useClass: SentryGlobalFilter,
     },
     {
       provide: Logger,

packages/nestjs/src/setup.ts

@@ -115,41 +115,6 @@ class SentryGlobalFilter extends BaseExceptionFilter {
 Catch()(SentryGlobalFilter);
 export { SentryGlobalFilter };
 
-/**
- * Global filter to handle exceptions in NestJS + GraphQL applications and report them to Sentry.
- *
- * @deprecated `SentryGlobalGraphQLFilter` is deprecated. Use the `SentryGlobalFilter` instead. The `SentryGlobalFilter` is a drop-in replacement.
- */
-class SentryGlobalGraphQLFilter {
-  private static readonly _logger = new Logger('ExceptionsHandler');
-  public readonly __SENTRY_INTERNAL__: boolean;
-
-  public constructor() {
-    this.__SENTRY_INTERNAL__ = true;
-  }
-
-  /**
-   * Catches exceptions and reports them to Sentry unless they are HttpExceptions.
-   */
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  public catch(exception: unknown, host: ArgumentsHost): void {
-    // neither report nor log HttpExceptions
-    if (exception instanceof HttpException) {
-      throw exception;
-    }
-    if (exception instanceof Error) {
-      // eslint-disable-next-line deprecation/deprecation
-      SentryGlobalGraphQLFilter._logger.error(exception.message, exception.stack);
-    }
-    captureException(exception);
-    throw exception;
-  }
-}
-// eslint-disable-next-line deprecation/deprecation
-Catch()(SentryGlobalGraphQLFilter);
-// eslint-disable-next-line deprecation/deprecation
-export { SentryGlobalGraphQLFilter };
-
 /**
  * Global filter to handle exceptions and report them to Sentry.
  *