bug fixes

App/Users/Model.py

@@ -56,7 +56,7 @@ class User(models.Model):
         """Hashes a plain password."""
         return pwd_context.hash(plain_password)
 
-    def verify_password(self, plain_password: str) -> bool:
+    async def verify_password(self, plain_password: str) -> bool:
         """
         Verifies a plain password against the hashed password.
         Handles account locking after multiple failed attempts.
@@ -75,14 +75,14 @@ class User(models.Model):
             self.failed_attempts = 0
             self.account_locked = False
             self.lastLogin = datetime.datetime.now()
-            self.save()
+            await self.save()
             return True
         else:
             self.failed_attempts += 1
             if self.failed_attempts >= 5:
                 self.account_locked = True
                 logger.warning(f"Account {self.phoneNumber} has been locked.")
-            self.save()
+            await self.save()
             return False
 
     async def initiate_password_reset(self):
@@ -257,3 +257,6 @@ class User(models.Model):
             logger.error(
                 f"Failed to send insufficient funds message to {self.phoneNumber}: {e}"
             )
+
+    async def delete_user(self):
+        pass

App/Users/UserRoutes.py

@@ -64,9 +64,9 @@ async def register_user(request: RegisterUserRequest):
 async def login_user(request: LoginUserRequest):
     # Find user by phone number
     db_user = await User.filter(phoneNumber=request.phoneNumber).first()
-
+    valid_password = await db_user.verify_password(request.password)
     # Check if user exists and password is correct
-    if db_user and db_user.verify_password(request.password):
+    if db_user and valid_password:
         # Fetch active subscription if it exists
         subscription = await Subscription.filter(user=db_user, active=True).first()