Merge pull request #134 from kauandotnet/hotfix/validate-active-user

fix: await promise before check the result
conceptadev · Oct 30, 2023 · 858bcb9 · 858bcb9
2 parents 4dccb93 + 8ccc400
commit 858bcb9
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/nestjs-auth-local/src/services/auth-local-validate-user.service.ts b/packages/nestjs-auth-local/src/services/auth-local-validate-user.service.ts
@@ -38,8 +38,10 @@ export class AuthLocalValidateUserService
       throw new Error(`No user found for username: ${dto.username}`);
     }
 
+    const isUserActive = await this.isActive(user);
+
     // is the user active?
-    if (!this.isActive(user)) {
+    if (!isUserActive) {
       throw new Error(`User with username '${dto.username}' is inactive`);
     }