Skip to content

Commit

Permalink
Bug fix: ignore rate throttling with 404s
Browse files Browse the repository at this point in the history
  • Loading branch information
martin-village committed Sep 28, 2024
1 parent 33a6fbe commit 96197ae
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tap_github/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,9 @@ def rate_throttling(response):
LOGGER.info("API rate limit exceeded. Tap will retry the data collection after %s seconds.", seconds_to_sleep)
# Arbitrary 10 second wait
time.sleep(10)
return True
if response.status_code == 404:
return False

# Raise an exception if `X-RateLimit-Remaining` is not found in the header.
# API does include this key header if provided base URL is not a valid github custom domain.
Expand Down

0 comments on commit 96197ae

Please sign in to comment.