-
Notifications
You must be signed in to change notification settings - Fork 25
Open
Description
Current rate-limiting is static, relying on https://www.npmjs.com/package/bottleneck, instead of dynamically throttling depending on the X-Rate-Limit-Remaining header on the response.
Implementing proper rate-limiting will:
- Improve performance, potentially dramatically.
- Reduce chance of accidentally hitting the rate limit when making batch changes and causing an absolute headache because it becomes difficult to determine which requests succeeded and which requests failed (as happened recently to me when running https://github.com/ubccapico/restore-speedgrader-comments.
Metadata
Metadata
Assignees
Labels
No labels