Added Support for NestJS 9 and updated README.md

Author: xiifain

.gitignore

@@ -1 +1,3 @@
 node_modules
+dist/*
+coverage/*

README.MD

@@ -51,7 +51,6 @@ The simplest way to use `@ntegral/nestjs-sentry` is to use `SentryModule.forRoot
 ```typescript
 import { Module } from '@nestjs-common';
 import { SentryModule } from '@ntegral/nestjs-sentry';
-import { LogLevel } from '@sentry/types';
 
 @Module({
   imports: [
@@ -60,7 +59,7 @@ import { LogLevel } from '@sentry/types';
       debug: true | false,
       environment: 'dev' | 'production' | 'some_environment',
       release: 'some_release', | null, // must create a release in sentry.io dashboard
-      logLevel: LogLevel.Debug //based on sentry.io loglevel //
+      logLevels: ['debug'] //based on sentry.io loglevel //
     }),
   ],
 })
@@ -84,7 +83,7 @@ import { ConfigService } from '@ntegral/nestjs-config';
         debug: true | false,
         environment: 'dev' | 'production' | 'some_environment',
         release: 'some_release', | null, // must create a release in sentry.io dashboard
-        logLevel: LogLevel.Debug //based on sentry.io loglevel //
+        logLevels: ['debug'] //based on sentry.io loglevel //
       }),
       inject: [ConfigService],
     })
@@ -128,13 +127,12 @@ You can instuct use the SentryService to log non-error messages as breadcrumbs o
 ```typescript
 import { Injectable } from '@nestjs/common';
 import { InjectSentry, SentryService } from '@ntegral/nestjs-sentry';
-import {  Severity } from '@sentry/types';
 
 @Injectable()
 export class AppService {
   constructor(@InjectSentry() private readonly client: SentryService) {
     client.log('AppSevice Loaded','test', true); // creates log asBreadcrumb //
-    client.instance().addBreadcrumb({level: Severity.Debug, message: 'How to use native breadcrumb', data: { context: 'WhatEver'}})
+    client.instance().addBreadcrumb({level: 'debug' , message: 'How to use native breadcrumb', data: { context: 'WhatEver'}})
     client.debug('AppService Debug', 'context');
   }
   getHello(): string {
@@ -169,6 +167,29 @@ import { GraphqlInterceptor } from '@ntegral/nestjs-sentry';
 export class AppModule {}
 ```
 
+Using the sentry interceptor globally
+```typescript
+import { Module } from '@nestjs/common';
+import { APP_INTERCEPTOR } from '@nestjs/core';
+import { SentryInterceptor } from '@ntegral/nestjs-sentry';
+
+@Module({
+  ....
+  providers: [
+    {
+      provide: APP_INTERCEPTOR,
+      useFactory: () => new SentryInterceptor({
+        filters: [{
+          type: HttpException,
+          filter: (exception: HttpException) => 500 > exception.getStatus() // Only report 500 errors
+        }]
+      }),
+    }
+  ],
+})
+export class AppModule {}
+```
+
 ## Flushing sentry
 Sentry does not flush all the errors by itself, it does it in background so that it doesn't block the main thread. If 
 you kill the nestjs app forcefully some exceptions don't have to be flushed and logged successfully.
@@ -180,7 +201,6 @@ for closing (flushing) to work.
 ```typescript
 import { Module } from '@nestjs-common';
 import { SentryModule } from '@ntegral/nestjs-sentry';
-import { LogLevel } from '@sentry/types';
 
 @Module({
   imports: [
@@ -189,7 +209,7 @@ import { LogLevel } from '@sentry/types';
       debug: true | false,
       environment: 'dev' | 'production' | 'some_environment',
       release: 'some_release', | null, // must create a release in sentry.io dashboard
-      logLevel: LogLevel.Debug //based on sentry.io loglevel //
+      logLevels: ['debug'] //based on sentry.io loglevel //
       close: {
         enabled: true,
         // Time in milliseconds to forcefully quit the application  

lib/__tests__/sentry.decorator.spec.ts

@@ -1,5 +1,4 @@
 import { Test, TestingModule } from '@nestjs/testing';
-import { LogLevel } from "@sentry/types";
 import { SentryModuleOptions } from '../sentry.interfaces';
 import { Injectable } from '@nestjs/common';
 import { InjectSentry } from '../sentry.decorator';
@@ -12,7 +11,7 @@ describe('InjectS3', () => {
         dsn: 'https://[email protected]/25956308132020',
         debug: true,
         environment: 'development',
-        logLevel: LogLevel.Debug,
+        logLevels: ['debug'],
     }
     let module: TestingModule;
 

lib/__tests__/sentry.module.spec.ts

@@ -2,7 +2,6 @@ import { Test } from '@nestjs/testing';
 
 import { SentryModule  } from '../sentry.module';
 import { SentryModuleOptions, SentryOptionsFactory } from '../sentry.interfaces';
-import { LogLevel } from '@sentry/types';
 import { SentryService } from '../sentry.service';
 import { SENTRY_TOKEN } from '../sentry.constants';
 import { Module } from '@nestjs/common';
@@ -12,7 +11,7 @@ describe('SentryModule', () => {
         dsn: 'https://[email protected]/25956308132020',
         debug: true,
         environment: 'development',
-        logLevel: LogLevel.Debug,
+        logLevels: ['debug'],
     }
 
     class TestService implements SentryOptionsFactory {

lib/__tests__/sentry.service.spec.ts

@@ -1,5 +1,4 @@
 import { SentryModuleOptions, SentryOptionsFactory } from "../sentry.interfaces";
-import { LogLevel } from "@sentry/types";
 import { Test, TestingModule } from "@nestjs/testing";
 import { SentryModule } from "../sentry.module";
 import { SentryService } from "../sentry.service";
@@ -18,14 +17,14 @@ describe('SentryService', () => {
         dsn: 'https://[email protected]/25956308132020',
         debug: true,
         environment: 'development',
-        logLevel: LogLevel.Debug,
+        logLevels: ['debug'],
     };
 
     let failureConfig: SentryModuleOptions = {
         dsn: 'https://[email protected]/1512xxx',
         debug: true,
         environment: 'development',
-        logLevel: LogLevel.Debug,
+        logLevels: ['debug'],
     };
 
     class TestService implements SentryOptionsFactory {

lib/sentry.interceptor.ts

@@ -2,6 +2,7 @@
 import {
   CallHandler,
   ExecutionContext,
+  HttpException,
   Injectable,
   NestInterceptor
 } from '@nestjs/common';
@@ -33,17 +34,17 @@ export class SentryInterceptor implements NestInterceptor {
   intercept(context: ExecutionContext, next: CallHandler): Observable<any> {
     // first param would be for events, second is for errors
     return next.handle().pipe(
-      tap(null, (exception) => {
+      tap(null, (exception : HttpException) => {
         if(this.shouldReport(exception)) {
           this.client.instance().withScope((scope) => {
-            this.captureException(context, scope, exception);
+            return this.captureException(context, scope, exception);
           })
         }
       })
     );
   }
 
-  protected captureException(context: ExecutionContext, scope: Scope, exception: any) {
+  protected captureException(context: ExecutionContext, scope: Scope, exception: HttpException) {
     switch (context.getType<ContextType>()) {
       case 'http':
         return this.captureHttpException(
@@ -66,7 +67,7 @@ export class SentryInterceptor implements NestInterceptor {
     }
   }
 
-  private captureHttpException(scope: Scope, http: HttpArgumentsHost, exception: any): void {
+  private captureHttpException(scope: Scope, http: HttpArgumentsHost, exception: HttpException): void {
     const data = Handlers.parseRequest(<any>{},http.getRequest(), this.options);
 
     scope.setExtra('req', data.request);

lib/sentry.interfaces.ts

@@ -1,7 +1,7 @@
 import { ModuleMetadata, Type } from "@nestjs/common/interfaces";
 import { Integration, Options } from '@sentry/types';
-import { Severity } from "@sentry/node";
-import { ConsoleLoggerOptions } from "@nestjs/common";
+import { ConsoleLoggerOptions, HttpException } from "@nestjs/common";
+import { SeverityLevel } from "@sentry/node";
 
 export interface SentryCloseOptions {
     enabled: boolean;
@@ -41,7 +41,7 @@ export interface SentryInterceptorOptions {
     tags?: { [key: string]: string };
     extra?: { [key: string]: any };
     fingerprint?: string[];
-    level?: Severity;
+    level?: SeverityLevel;
 
     // https://github.com/getsentry/sentry-javascript/blob/master/packages/node/src/handlers.ts#L163
     request?: boolean;

lib/sentry.service.ts

@@ -1,6 +1,6 @@
 import { Inject, Injectable, ConsoleLogger } from '@nestjs/common';
 import { OnApplicationShutdown } from '@nestjs/common';
-import { Options, Client } from '@sentry/types';
+import { ClientOptions, Client } from '@sentry/types';
 import * as Sentry from '@sentry/node';
 import { SENTRY_MODULE_OPTIONS } from './sentry.constants';
 import { SentryModuleOptions } from './sentry.interfaces';
@@ -31,8 +31,8 @@ export class SentryService extends ConsoleLogger implements OnApplicationShutdow
             } else {
               (
                 Sentry.getCurrentHub().getClient<
-                  Client<Options>
-                >() as Client<Options>
+                  Client<ClientOptions>
+                >() as Client<ClientOptions>
               ).captureException(err);
               process.exit(1);
             }
@@ -58,20 +58,20 @@ export class SentryService extends ConsoleLogger implements OnApplicationShutdow
       asBreadcrumb ?
       Sentry.addBreadcrumb({
         message,
-        level: Sentry.Severity.Log,
+        level: 'log',
         data: {
           context
         }
       }) :
-      Sentry.captureMessage(message, Sentry.Severity.Log);
+      Sentry.captureMessage(message, 'log');
     } catch (err) {}
   }
 
   error(message: string, trace?: string, context?: string) {
     message = `${this.app} ${message}`;
     try {
       super.error(message, trace, context);
-      Sentry.captureMessage(message, Sentry.Severity.Error);
+      Sentry.captureMessage(message, 'error');
     } catch (err) {}
   }
 
@@ -82,12 +82,12 @@ export class SentryService extends ConsoleLogger implements OnApplicationShutdow
       asBreadcrumb ?
       Sentry.addBreadcrumb({
         message,
-        level: Sentry.Severity.Warning,
+        level: 'warning',
         data: {
           context
         }
       }) :
-      Sentry.captureMessage(message, Sentry.Severity.Warning);
+      Sentry.captureMessage(message, 'warning');
     } catch (err) {}
   }
 
@@ -98,12 +98,12 @@ export class SentryService extends ConsoleLogger implements OnApplicationShutdow
       asBreadcrumb ?
       Sentry.addBreadcrumb({
         message,
-        level: Sentry.Severity.Debug,
+        level: 'debug',
         data: {
           context
         }
       }) :
-      Sentry.captureMessage(message, Sentry.Severity.Debug);
+      Sentry.captureMessage(message, 'debug');
     } catch (err) {}
   }
 
@@ -114,12 +114,12 @@ export class SentryService extends ConsoleLogger implements OnApplicationShutdow
       asBreadcrumb ?
       Sentry.addBreadcrumb({
         message,
-        level: Sentry.Severity.Info,
+        level: 'info',
         data: {
           context
         }
       }) :
-      Sentry.captureMessage(message, Sentry.Severity.Info);
+      Sentry.captureMessage(message, 'info');
     } catch (err) {}
   }