Replace grequests with asyncio solution (#1027)

* Draft pass at asyncio replacement for grequests * Use async_to_sync to ensure Django safety * Update unit tests for asyncio validate_images * Add back 2s timeout and do not follow redirects * Use better test assertion Co-authored-by: Madison Swain-Bowden <[email protected]> * Add return type annotations and clean up dead code * Fix suggested code typo * Add type annotations and switch to comprehension Co-authored-by: Madison Swain-Bowden <[email protected]>
WordPress · Dec 15, 2022 · 9b3059f · 9b3059f
1 parent 6b75766
commit 9b3059f
Show file tree

Hide file tree

Showing 5 changed files with 488 additions and 136 deletions.
diff --git a/api/Pipfile b/api/Pipfile
@@ -18,6 +18,7 @@ pytest-raises = "~=0.11"
 remote-pdb = "~=2.1"
 sphinx = "~=5.2"
 sphinx-autobuild = "~=2021.3"
+pook = "~=1.0"
 
 [packages]
 aws-requests-auth = "~=0.4"
@@ -40,7 +41,6 @@ drf-yasg = "~=1.21"
 elasticsearch-dsl = "~=7.4"
 future = "~=0.18"
 gevent = "~=22.10"
-grequests = "~=0.6"
 gunicorn = "~=20.1"
 hvac = "~=1.0"
 ipaddress = "~=1.0"
@@ -56,6 +56,7 @@ redlock-py = "~=1.0"
 requests-oauthlib = "~=1.3"
 sentry-sdk = "~=1.11"
 wsgi-basic-auth = "~=1.1"
+aiohttp = "~=3.8"
 
 [requires]
 python_version = "3.10"