Update modules/auth/auth.py

modules/auth/auth.py

@@ -23,6 +23,7 @@ from datetime import datetime, timezone
 
 logger = logging.getLogger(__name__)
 
+#####################################################################
 def clean_and_validate_key(key):
     """Limpia y valida la clave de CosmosDB"""
     key = key.strip()
@@ -43,7 +44,7 @@ if not endpoint or not key:
 
 key = clean_and_validate_key(key)
 
-
+#####################################################################
 def authenticate_user(username, password):
     """Autentica un usuario y registra el inicio de sesión"""
     try:
@@ -77,6 +78,7 @@ def authenticate_user(username, password):
         logger.error(f"Error durante la autenticación del usuario: {str(e)}")
         return False, None
 
+#####################################################################
 def authenticate_student(username, password):
     """Autentica un estudiante"""
     success, role = authenticate_user(username, password)
@@ -84,6 +86,7 @@ def authenticate_student(username, password):
         return True, role
     return False, None
 
+    #####################################################################
 def authenticate_admin(username, password):
     """Autentica un administrador"""
     success, role = authenticate_user(username, password)
@@ -91,6 +94,7 @@ def authenticate_admin(username, password):
         return True, role
     return False, None
 
+#####################################################################
 def register_student(username, password, additional_info=None):
     """Registra un nuevo estudiante"""
     try:
@@ -98,6 +102,7 @@ def register_student(username, password, additional_info=None):
             logger.warning(f"Estudiante ya existe: {username}")
             return False
 
+        # Aquí está el problema - debemos hashear la contraseña ANTES de pasarla
         hashed_password = hash_password(password)
         
         # Asegurarse que additional_info tenga el rol correcto
@@ -105,6 +110,7 @@ def register_student(username, password, additional_info=None):
             additional_info = {}
         additional_info['role'] = 'Estudiante'
         
+        # Pasar la contraseña hasheada, no la original
         success = create_student_user(username, hashed_password, additional_info)
         if success:
             logger.info(f"Nuevo estudiante registrado: {username}")
@@ -117,6 +123,8 @@ def register_student(username, password, additional_info=None):
         logger.error(f"Error al registrar estudiante: {str(e)}")
         return False
 
+        
+        #####################################################################
 def update_student_info(username, new_info):
     """Actualiza la información de un estudiante"""
     try:
@@ -135,6 +143,7 @@ def update_student_info(username, new_info):
         logger.error(f"Error en actualización: {str(e)}")
         return False
 
+#####################################################################
 def delete_student(username):
     """Elimina un estudiante"""
     try:
@@ -150,6 +159,7 @@ def delete_student(username):
         logger.error(f"Error en eliminación: {str(e)}")
         return False
 
+        #####################################################################
 def logout():
     """Cierra la sesión del usuario"""
     try:
@@ -171,15 +181,28 @@ def logout():
 ########################################################
 def hash_password(password):
     """Hashea una contraseña usando bcrypt"""
-    # Usar 12 rondas para el salt, que es lo que se ve en el hash proporcionado
-    return bcrypt.hashpw(
-        password.encode('utf-8'), 
-        bcrypt.gensalt(rounds=12)
-    ).decode('utf-8')
+    try:
+        # Usar 12 rondas para el salt
+        hashed = bcrypt.hashpw(
+            password.encode('utf-8'),
+            bcrypt.gensalt(rounds=12)
+        ).decode('utf-8')
+        
+        # Verificar que el hash tenga el formato correcto
+        if not hashed.startswith('$2b$12$'):
+            logger.error(f"Hash generado con formato incorrecto: {hashed[:10]}...")
+            raise ValueError("Hash generado con formato incorrecto")
+            
+        logger.info(f"Contraseña hasheada exitosamente. Hash comienza con: {hashed[:10]}...")
+        return hashed
+        
+    except Exception as e:
+        logger.error(f"Error hasheando contraseña: {str(e)}")
+        raise
 ########################################################
 
     
-
+#####################################################################
 def verify_password(stored_password, provided_password):
     """Verifica una contraseña"""
     return bcrypt.checkpw(