Update modules/auth/auth.py

modules/auth/auth.py

@@ -23,7 +23,6 @@ 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()
@@ -44,7 +43,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:
@@ -78,7 +77,6 @@ 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)
@@ -86,7 +84,6 @@ 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)
@@ -94,7 +91,6 @@ def authenticate_admin(username, password):
         return True, role
     return False, None
 
-#####################################################################
 def register_student(username, password, additional_info=None):
     """Registra un nuevo estudiante"""
     try:
@@ -102,7 +98,6 @@ 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
@@ -110,7 +105,6 @@ 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}")
@@ -123,8 +117,6 @@ 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:
@@ -143,7 +135,6 @@ 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:
@@ -159,7 +150,6 @@ def delete_student(username):
         logger.error(f"Error en eliminación: {str(e)}")
         return False
 
-        #####################################################################
 def logout():
     """Cierra la sesión del usuario"""
     try:
@@ -178,31 +168,13 @@ def logout():
     finally:
         st.session_state.clear()
 
-########################################################
 def hash_password(password):
-    """Hashea una contraseña usando bcrypt"""
-    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
-########################################################
+    """Hashea una contraseña"""
+    return bcrypt.hashpw(
+        password.encode('utf-8'), 
+        bcrypt.gensalt()
+    ).decode('utf-8')
 
-    
-#####################################################################
 def verify_password(stored_password, provided_password):
     """Verifica una contraseña"""
     return bcrypt.checkpw(