fix: add status code 429
to bandh backOffStatusCodes
#404
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
processBackoffDelay only triggers with status codes 403.
I was getting a lot of RATE LIMIT EXCEEDEDs from B&H.
In lookupCard, however, it does check for 429 and logs it as a warning.
I figure that getting these RATE LIMIT EXCEEDEDs is bad and it should trigger a backoff, so I added 429 to bandh.ts to the backoffStatusCodes: [403, 429] array/list.
A lot less RATE LIMIT EXCEEDEDs
Testing
I've been running it with the added 429 and it's been running fine for the last few hours with no issues. Less RATE LIMIT EXCEEDEDs.
edit: code formatting