Merge pull request #355 from gilesknap/http429

HTTP 429
gilesknap · May 17, 2022 · 94c6a25 · 94c6a25
2 parents 3b986fa + a8cefbc
commit 94c6a25
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/code.yml b/.github/workflows/code.yml
@@ -64,10 +64,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: ["ubuntu-latest", "windows-latest"] # can add , macos-latest
+        os: ["ubuntu-latest", "windows-latest", "macos-latest"]
         # keep matrix tight for now to avoid HTTP 429 errors
         # TODO: restore macos and 3.9 and 3.10
-        python: ["3.8"]
+        python: ["3.8", "3.9"]
         lock: [false]
 
         include:

diff --git a/src/gphotos_sync/GooglePhotosDownload.py b/src/gphotos_sync/GooglePhotosDownload.py
@@ -88,7 +88,7 @@ def __init__(
         retries = Retry(
             total=settings.max_retries,
             backoff_factor=5,
-            status_forcelist=[500, 502, 503, 504, 509],
+            status_forcelist=[500, 502, 503, 504, 509, 429],
             allowed_methods=frozenset(["GET", "POST"]),
             raise_on_status=False,
             respect_retry_after_header=True,

diff --git a/src/gphotos_sync/authorize.py b/src/gphotos_sync/authorize.py
@@ -112,7 +112,7 @@ def authorize(self):
         retries = Retry(
             total=self.max_retries,
             backoff_factor=5,
-            status_forcelist=[500, 502, 503, 504],
+            status_forcelist=[500, 502, 503, 504, 429],
             allowed_methods=frozenset(["GET", "POST"]),
             raise_on_status=False,
             respect_retry_after_header=True,