New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Meraki - Enable API call rate limiting for requests #54827
Conversation
The test
The test
|
The test
The test
The test
|
- Detects if error code is 429 - Pauses for random time between .5 and 5 seconds before retrying - If it fails 10 times, give up and tell user
- 3 * number of retries - Fails after the 30 second wait
- Errors aren't tested but code works for regular calls
c80a0bd
to
38e8c35
Compare
The test
The test
The test
|
* Initial commit for rate limiting - Detects if error code is 429 - Pauses for random time between .5 and 5 seconds before retrying - If it fails 10 times, give up and tell user * Redo structure of request() to support rate limiting * Hold down timer is now a sliding scale - 3 * number of retries - Fails after the 30 second wait * Whitespace fixes * Redo implementation using decorators - Errors aren't tested but code works for regular calls * Unit tests work for error handling * Add integration tests for successful retries * Add condition for 502 errors and retry * Move _error_report out of the class * PEP8 fixes * Add changelog entry
* Initial commit for rate limiting - Detects if error code is 429 - Pauses for random time between .5 and 5 seconds before retrying - If it fails 10 times, give up and tell user * Redo structure of request() to support rate limiting * Hold down timer is now a sliding scale - 3 * number of retries - Fails after the 30 second wait * Whitespace fixes * Redo implementation using decorators - Errors aren't tested but code works for regular calls * Unit tests work for error handling * Add integration tests for successful retries * Add condition for 502 errors and retry * Move _error_report out of the class * PEP8 fixes * Add changelog entry
SUMMARY
ISSUE TYPE
COMPONENT NAME
meraki