From ec8eaf52bfdc2bde612e4fc0324575ba7dc6b2e1 Mon Sep 17 00:00:00 2001 From: Fringg Date: Tue, 10 Feb 2026 07:27:20 +0300 Subject: [PATCH] fix: downgrade transient API errors (502/503/504) to warning level 502/503/504 are transient errors that don't need ERROR reports in chat. Also downgrade API connection test failure to warning. --- app/external/remnawave_api.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/external/remnawave_api.py b/app/external/remnawave_api.py index 63293186..087beb04 100644 --- a/app/external/remnawave_api.py +++ b/app/external/remnawave_api.py @@ -400,8 +400,9 @@ class RemnaWaveAPI: if response.status >= 400: error_message = response_data.get('message', f'HTTP {response.status}') - logger.error(f'API Error {response.status}: {error_message}') - logger.error(f'Response: {response_text[:500]}') + log = logger.warning if response.status in (502, 503, 504) else logger.error + log('API Error %s: %s', response.status, error_message) + log('Response: %s', response_text[:500]) raise RemnaWaveAPIError(error_message, response.status, response_data) return response_data @@ -1284,5 +1285,5 @@ async def test_api_connection(api: RemnaWaveAPI) -> bool: await api.get_system_stats() return True except Exception as e: - logger.error(f'API connection test failed: {e}') + logger.warning('API connection test failed: %s', e) return False