Retry aiohttp requests on asyncio.TimeoutError (#701)

Author: droserasprout

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog], and this project adheres to [Semantic Versioning].
 
+## [Unreleased]
+
+### Fixed
+
+- http: Mark `asyncio.TimeoutError` exception as safe to retry.
+
 ## [6.5.6] - 2022-05-02
 
 ### Fixed

README.md

@@ -60,6 +60,7 @@ We are grateful to all the people who help us with the project.
 - [@sbihel](https://github.com/sbihel)
 - [@tezosmiami](https://github.com/tezosmiami)
 - [@tomsib2001](https://github.com/tomsib2001)
+- [@TristanAllaire](https://github.com/TristanAllaire)
 - [@veqtor](https://github.com/veqtor)
 - [@xflpt](https://github.com/xflpt)
 

src/dipdup/http.py

@@ -26,11 +26,13 @@
 from dipdup.exceptions import InvalidRequestError
 from dipdup.prometheus import Metrics
 
-safe_exceptions = (
+exceptions_to_retry = (
     aiohttp.ClientConnectionError,
     aiohttp.ClientConnectorError,
     aiohttp.ClientResponseError,
     aiohttp.ClientPayloadError,
+    # NOTE: aiohttp doesn't reraise it
+    asyncio.TimeoutError,
 )
 
 
@@ -149,7 +151,7 @@ async def _retry_request(
                     weight=weight,
                     **kwargs,
                 )
-            except safe_exceptions as e:
+            except exceptions_to_retry as e:
                 if self._config.retry_count and attempt - 1 == self._config.retry_count:
                     raise e