feat(nestjs): Filter 4xx errors

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

@@ -45,6 +45,11 @@ export class AppController1 {
     return this.appService.testException(id);
   }
 
+  @Get("test-expected-exception/:id")
+  async testExpectedException(@Param("id") id: string) {
+    return this.appService.testExpectedException(id);
+  }
+
   @Get('test-outgoing-fetch-external-allowed')
   async testOutgoingFetchExternalAllowed() {
     return this.appService.testOutgoingFetchExternalAllowed();

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

@@ -1,4 +1,4 @@
-import { Injectable } from '@nestjs/common';
+import { HttpException, HttpStatus, Injectable } from '@nestjs/common';
 import * as Sentry from '@sentry/nestjs';
 import { makeHttpRequest } from './utils';
 
@@ -52,6 +52,10 @@ export class AppService1 {
     throw new Error(`This is an exception with id ${id}`);
   }
 
+  testExpectedException(id: string) {
+    throw new HttpException(`This is an expected exception with id ${id}`, HttpStatus.FORBIDDEN);
+  }
+
   async testOutgoingFetchExternalAllowed() {
     const fetchResponse = await fetch('http://localhost:3040/external-allowed');
 

dev-packages/e2e-tests/test-applications/nestjs/tests/errors.test.ts

@@ -11,6 +11,9 @@ test('Sends exception to Sentry', async ({ baseURL }) => {
 
   const errorEvent = await errorEventPromise;
 
+  console.log("UNEXPECTED EXCEPTION");
+  console.log(errorEvent);
+
   expect(errorEvent.exception?.values).toHaveLength(1);
   expect(errorEvent.exception?.values?.[0]?.value).toBe('This is an exception with id 123');
 
@@ -28,3 +31,19 @@ test('Sends exception to Sentry', async ({ baseURL }) => {
     span_id: expect.any(String),
   });
 });
+
+test('Does not send expected exception to Sentry', async ({ baseURL }) => {
+  const errorEventPromise = waitForError('nestjs', event => {
+    return !event.type && event.exception?.values?.[0]?.value === 'This is an unexpected exception with id 123';
+  });
+
+  const response = await fetch(`${baseURL}/test-expected-exception/123`);
+  expect(response.status).toBe(403);
+
+  const errorEvent = await errorEventPromise;
+
+  console.log("EXPECTED EXCEPTION");
+  console.log(errorEvent);
+
+  expect(errorEvent.exception?.values).toHaveLength(1);
+});