Skip to content

Commit

Permalink
Merge branch 'develop' into prepare-release/8.28.0
Browse files Browse the repository at this point in the history
  • Loading branch information
nicohrubec committed Sep 3, 2024
2 parents e8ed7c2 + 9b8f9ea commit 9e585e7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -908,6 +908,7 @@ jobs:
'nestjs-basic',
'nestjs-distributed-tracing',
'nestjs-with-submodules',
'nestjs-with-submodules-decorator',
'nestjs-graphql',
'node-exports-test-app',
'node-koa',
Expand Down
4 changes: 2 additions & 2 deletions packages/nestjs/src/decorators/with-sentry.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,11 @@ export function WithSentry() {
// eslint-disable-next-line @typescript-eslint/no-explicit-any
descriptor.value = function (exception: unknown, host: unknown, ...args: any[]) {
if (isExpectedError(exception)) {
return originalCatch.apply(this, args);
return originalCatch.apply(this, [exception, host, ...args]);
}

captureException(exception);
return originalCatch.apply(this, args);
return originalCatch.apply(this, [exception, host, ...args]);
};

return descriptor;
Expand Down

0 comments on commit 9e585e7

Please sign in to comment.