Polykey
Polykey copied to clipboard
Optimise the performance of IdPs
Created by @CMCDragonkai
The following discussion from !141 should be addressed:
-
[ ] @CMCDragonkai started a discussion: (+2 comments)
Need to keep a note here regarding API limits and performance optimisation of the providers.
- https://docs.github.com/en/rest/overview/resources-in-the-rest-api#rate-limiting
- https://docs.github.com/en/rest/overview/resources-in-the-rest-api#conditional-requests
There's a 5K API request per hour limit shared across all oauth2 apps on a particualar user. We can keep track of the limits as well. So when we get throttled we understand we cannot contact the provider API. This should be presented to the user to prevent frustration.
So basically:
- Resource caching
- Rate limit tracking
- Conditional requests (based on resource caching)
should all be applied where we can do so.