From dbcf962f6d57d2c2096d932466b75952253208a8 Mon Sep 17 00:00:00 2001 From: Gujiassh Date: Thu, 5 Mar 2026 10:28:51 +0900 Subject: [PATCH] refactor(sdk-python): hoist httpx network error names --- .../adapters/converter/exception_converter.py | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/sdks/sandbox/python/src/opensandbox/adapters/converter/exception_converter.py b/sdks/sandbox/python/src/opensandbox/adapters/converter/exception_converter.py index d235c64b..15d3ca96 100644 --- a/sdks/sandbox/python/src/opensandbox/adapters/converter/exception_converter.py +++ b/sdks/sandbox/python/src/opensandbox/adapters/converter/exception_converter.py @@ -40,6 +40,14 @@ logger = logging.getLogger(__name__) +HTTPX_NETWORK_ERROR_TYPES = { + "ConnectError", + "TimeoutException", + "NetworkError", + "ReadTimeout", + "WriteTimeout", +} + class ExceptionConverter: """ @@ -134,14 +142,7 @@ def _is_httpx_status_error(e: Exception) -> bool: def _is_httpx_network_error(e: Exception) -> bool: """Check if exception is an httpx network-related error.""" - error_types = ( - "ConnectError", - "TimeoutException", - "NetworkError", - "ReadTimeout", - "WriteTimeout", - ) - return type(e).__name__ in error_types + return type(e).__name__ in HTTPX_NETWORK_ERROR_TYPES def _convert_unexpected_status_to_api_exception(e: Exception) -> SandboxApiException: