refactor: drop the unreachable except aiohttp.ClientError: raise
T3-21: neither curl_cffi nor noble_tls raises aiohttp exceptions (verified in both packages' sources), so the branch could not run. every error a real backend raises maps exactly as before; the asyncio.TimeoutError guard stays (TimeoutError is an OSError on 3.11+). folds into 1.0.5. Signed-off-by: disqualifier <dev@disqualifier.me>
This commit is contained in:
1 parent
56a5db9c0c
commit
3012c40c27
1 file changed
+2
-4
@@ -172,9 +172,8 @@ class CurlCffi:
|
|||||||
|
|
||||||
try:
|
try:
|
||||||
response = await session.request(method, url, impersonate=impersonate, **kwargs)
|
response = await session.request(method, url, impersonate=impersonate, **kwargs)
|
||||||
except aiohttp.ClientError:
|
|
||||||
raise
|
|
||||||
except asyncio.TimeoutError:
|
except asyncio.TimeoutError:
|
||||||
|
# TimeoutError is an OSError (3.11+): keep it out of the wrap below
|
||||||
raise
|
raise
|
||||||
except OSError as error:
|
except OSError as error:
|
||||||
if _is_timeout_error(error):
|
if _is_timeout_error(error):
|
||||||
@@ -316,9 +315,8 @@ class Noble:
|
|||||||
|
|
||||||
try:
|
try:
|
||||||
response = await session.execute_request(method=method.upper(), url=url, **kwargs)
|
response = await session.execute_request(method=method.upper(), url=url, **kwargs)
|
||||||
except aiohttp.ClientError:
|
|
||||||
raise
|
|
||||||
except asyncio.TimeoutError:
|
except asyncio.TimeoutError:
|
||||||
|
# TimeoutError is an OSError (3.11+): keep it out of the wrap below
|
||||||
raise
|
raise
|
||||||
except OSError as error:
|
except OSError as error:
|
||||||
if _is_timeout_error(error):
|
if _is_timeout_error(error):
|
||||||
|
|||||||
Reference in new issue
Block a user