Conversation
…fineCachedFunction`
added timeout to the check release/changelogFile functions
Thanks for opening this pull request! 🎉We really appreciate you taking the time to contribute, @WilcoSp. A maintainer will take a look as soon as they can. In the meantime, please make sure that:
If anything needs adjusting we'll leave comments here. Thanks again! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
🔗 Linked issue
relates to #3263
🧭 Context
For resolving changelog info I've now wrapped most of the
checkfunctions indefineCachedFunctionto have the results cached and reduce fetching from the git providers.Also added timeouts to the last fetch calls for changelogs that didn't have timeout configured.
📚 Description
Due to Gitlab making their rate limits stricter I've now wrapped most of the
checkfunctions indefineCachedFunctionto have the result cached, reduce fetch calls & reduce chances of rate limits.I've made the change for all git providers to also prevent issues with other git providers, except github with releases because ungh also caches the results and I don't want old results to be hanging around too long.
I've configured the caching for the check functions to be have maxAge at 1 hour & staleMaxAge at 30 minutes, this is to prevent having stale data being served while the endpoint cache is being renewed.
I didn't make changes for the other endpoints because they're already covered by
defineCachedHandler.I've also added timeout to the last fetch calls that changelog uses to prevent hanging when a git providers take a long time or is being overloaded.