From 4c206293b1370b5eada7f01e417fba3939fd70cf Mon Sep 17 00:00:00 2001 From: ajaysi Date: Tue, 31 Mar 2026 19:38:54 +0530 Subject: [PATCH] Fix error handling in main_text_generation.py - Add HTTPException re-raise before generic Exception handler - Use static error message instead of str(e) which was out of scope - Fixes 'e is not associated with a value' error --- backend/services/llm_providers/main_text_generation.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/backend/services/llm_providers/main_text_generation.py b/backend/services/llm_providers/main_text_generation.py index c856bc38..47d4a1d0 100644 --- a/backend/services/llm_providers/main_text_generation.py +++ b/backend/services/llm_providers/main_text_generation.py @@ -509,7 +509,7 @@ def llm_text_gen( status_code=429, detail={ "error": "All LLM providers failed", - "message": str(e), + "message": "All configured LLM providers failed to generate a response. Please check API keys and try again.", "usage_info": { "error_type": "all_providers_failed", "operation_type": "text-generation", @@ -522,6 +522,9 @@ def llm_text_gen( } ) + except HTTPException: + # Re-raise HTTPExceptions (e.g., 429 subscription limit) - preserve error details + raise except Exception as e: logger.error(f"[llm_text_gen] Error during text generation: {str(e)}") raise