eliminar fallos de asyncio
This commit is contained in:
@@ -88,15 +88,49 @@ async def obtener_remesa(**kwargs) -> Dict[str, Any]:
|
||||
# Generar nombre de archivo
|
||||
file_name = f"vu_RM_{pedimento_data.get('pedimento_app', 'unknown')}.xml"
|
||||
if soap_error(soap_response):
|
||||
file_name = f"vu_RM_{pedimento_data.get('pedimento_app', 'unknown')}_ERROR.xml"
|
||||
document_response = await remesa_rest_controller.post_document(
|
||||
soap_response=soap_response,
|
||||
organizacion=pedimento_data.get('organizacion'),
|
||||
pedimento=pedimento_data.get('id'),
|
||||
file_name=file_name,
|
||||
document_type=16,
|
||||
)
|
||||
raise HTTPException(status_code=500, detail="Error en la respuesta del servicio SOAP")
|
||||
|
||||
file_name = f"vu_RM_{pedimento_data.get('pedimento_app', 'unknown')}_ERROR.xml"
|
||||
document_response = await remesa_rest_controller.post_document(
|
||||
soap_response=soap_response,
|
||||
organizacion=pedimento_data.get('organizacion'),
|
||||
pedimento=pedimento_data.get('id'),
|
||||
file_name=file_name,
|
||||
document_type=16,
|
||||
)
|
||||
|
||||
# Aquí necesitamos extraer el mensaje de error real
|
||||
error_message = "Error en la respuesta del servicio SOAP"
|
||||
|
||||
# Intentar extraer mensaje de error del XML de respuesta
|
||||
if hasattr(soap_response, 'text') and soap_response.text:
|
||||
try:
|
||||
import xml.etree.ElementTree as ET
|
||||
root = ET.fromstring(soap_response.text)
|
||||
|
||||
# Buscar mensajes de error comunes en respuestas SOAP de VUCEM
|
||||
# Esto puede variar según el servicio, pero comúnmente buscan:
|
||||
for fault in root.findall('.//{http://schemas.xmlsoap.org/soap/envelope/}Fault'):
|
||||
faultcode = fault.find('.//faultcode')
|
||||
faultstring = fault.find('.//faultstring')
|
||||
if faultstring is not None and faultstring.text:
|
||||
error_message = faultstring.text
|
||||
break
|
||||
|
||||
# También podría estar en una estructura de error específica de VUCEM
|
||||
for error in root.findall('.//{http://www.ventanillaunica.gob.mx/common/ws/oxml/respuesta}error'):
|
||||
msg = error.find('.//{http://www.ventanillaunica.gob.mx/common/ws/oxml/respuesta}message')
|
||||
if msg is not None and msg.text:
|
||||
error_message = msg.text
|
||||
break
|
||||
|
||||
except Exception as parse_error:
|
||||
logger.error(f"Error al parsear respuesta SOAP para extraer mensaje: {parse_error}")
|
||||
|
||||
# Lanzar excepción con el mensaje de error real
|
||||
raise HTTPException(
|
||||
status_code=500,
|
||||
detail=f"Error en la respuesta del servicio SOAP: {error_message}"
|
||||
)
|
||||
# Enviar documento
|
||||
try:
|
||||
|
||||
|
||||
@@ -17,13 +17,16 @@ def process_remesa_request(self, remesa_request: dict) -> dict:
|
||||
Returns:
|
||||
dict: Resultado del procesamiento con estado y detalles
|
||||
"""
|
||||
loop = asyncio.get_event_loop()
|
||||
task_id = self.request.id
|
||||
servicio = 5 # Código para Pedimento Remesas
|
||||
pedimento_id = remesa_request.get('pedimento', {}).get('id')
|
||||
organizacion_id = remesa_request.get('pedimento', {}).get('organizacion')
|
||||
remesa_num = remesa_request.get('remesa', 'N/A')
|
||||
|
||||
# Crear un NUEVO event loop para esta tarea (evita problemas de loop cerrado)
|
||||
loop = asyncio.new_event_loop()
|
||||
asyncio.set_event_loop(loop)
|
||||
|
||||
try:
|
||||
# Actualizar estado a processing
|
||||
loop.run_until_complete(
|
||||
@@ -49,7 +52,6 @@ def process_remesa_request(self, remesa_request: dict) -> dict:
|
||||
pedimento_id=pedimento_id,
|
||||
organizacion_id=organizacion_id,
|
||||
servicio=servicio,
|
||||
result=result
|
||||
)
|
||||
)
|
||||
|
||||
@@ -60,20 +62,43 @@ def process_remesa_request(self, remesa_request: dict) -> dict:
|
||||
|
||||
# Actualizar estado a failed
|
||||
try:
|
||||
loop.run_until_complete(
|
||||
update_task(
|
||||
task_id=task_id,
|
||||
message=f"Error al procesar remesa {remesa_num}: {str(e)}",
|
||||
status="failed",
|
||||
pedimento_id=pedimento_id,
|
||||
organizacion_id=organizacion_id,
|
||||
servicio=servicio,
|
||||
error=str(e)
|
||||
# Verificar si el loop aún está abierto
|
||||
if not loop.is_closed():
|
||||
loop.run_until_complete(
|
||||
update_task(
|
||||
task_id=task_id,
|
||||
message=f"Error al procesar remesa {remesa_num}: {str(e)}",
|
||||
status="failed",
|
||||
pedimento_id=pedimento_id,
|
||||
organizacion_id=organizacion_id,
|
||||
servicio=servicio,
|
||||
)
|
||||
)
|
||||
)
|
||||
else:
|
||||
# Si el loop está cerrado, crear uno nuevo temporal
|
||||
logger.warning(f"Loop cerrado, creando loop temporal para actualizar error")
|
||||
temp_loop = asyncio.new_event_loop()
|
||||
asyncio.set_event_loop(temp_loop)
|
||||
try:
|
||||
temp_loop.run_until_complete(
|
||||
update_task(
|
||||
task_id=task_id,
|
||||
message=f"Error al procesar remesa {remesa_num}: {str(e)}",
|
||||
status="failed",
|
||||
pedimento_id=pedimento_id,
|
||||
organizacion_id=organizacion_id,
|
||||
servicio=servicio
|
||||
)
|
||||
)
|
||||
finally:
|
||||
temp_loop.close()
|
||||
except Exception as update_error:
|
||||
logger.error(f"Error actualizando estado de tarea: {update_error}")
|
||||
|
||||
# Re-lanzar la excepción para que Celery la marque como fallida
|
||||
raise
|
||||
|
||||
|
||||
finally:
|
||||
# Limpiar el event loop
|
||||
if not loop.is_closed():
|
||||
loop.close()
|
||||
Reference in New Issue
Block a user