Merge pull request #130 from conceptadev/feature/jwt-exceptions

fix: auth jwt exception
conceptadev · Oct 24, 2023 · eb0f5ea · eb0f5ea
2 parents eaca7d8 + 8ff863e
commit eb0f5ea
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/packages/nestjs-auth-jwt/src/decorator/auth-jwt.guard.ts b/packages/nestjs-auth-jwt/src/decorator/auth-jwt.guard.ts
@@ -18,10 +18,14 @@ export class JwtAuthGuard extends AuthGuard(AUTH_JWT_STRATEGY_NAME) {
   }
 
   // handleRequest(err, user, info) {
-  handleRequest<T = ReferenceIdInterface>(err: Error | undefined, user: T) {
+  handleRequest<T = ReferenceIdInterface>(
+    err: Error | undefined,
+    user: T,
+    info?: Error,
+  ) {
     // You can throw an exception based on either "info" or "err" arguments
     if (err || !user) {
-      throw err || new UnauthorizedException();
+      throw new UnauthorizedException(null, { cause: err ?? info });
     }
     return user;
   }

diff --git a/packages/nestjs-jwt/src/jwt.strategy.spec.ts b/packages/nestjs-jwt/src/jwt.strategy.spec.ts
@@ -53,13 +53,12 @@ describe(JwtStrategy, () => {
     });
 
     it('should throw exception', async () => {
-      const t = async () => await jwtStrategy['verifyTokenCallback']();
+      const t = async () => jwtStrategy['verifyTokenCallback']();
       expect(t).rejects.toThrow();
     });
 
     it('should throw exception', async () => {
-      const t = async () =>
-        await jwtStrategy['verifyTokenCallback'](new Error());
+      const t = async () => jwtStrategy['verifyTokenCallback'](new Error());
       expect(t).rejects.toThrow();
     });