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.
This commit is contained in:
Fringg
2026-02-10 07:27:20 +03:00
parent fe5f5ded96
commit ec8eaf52bf

View File

@@ -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