-
Notifications
You must be signed in to change notification settings - Fork 30.5k
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
Tools updater throws 402 rate limit exceeded error #48119
Comments
I think its because the rate limit for unauthenticated github api calls is very low |
Exactly, the rate limit is 60 calls per hour for unauthenticated requests
The Job failed with the following error. To solve that problem we can authenticate the request to increase the rate limit to 1000 calls per hour, or read the
|
I guess we can authenticate the request |
PR-URL: #48200 Fixes: #48119 Refs: nodejs/security-wg#973 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Mestery <mestery@protonmail.com>
PR-URL: #48200 Fixes: #48119 Refs: nodejs/security-wg#973 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Mestery <mestery@protonmail.com>
PR-URL: nodejs#48200 Fixes: nodejs#48119 Refs: nodejs/security-wg#973 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Mestery <mestery@protonmail.com>
PR-URL: nodejs#48200 Fixes: nodejs#48119 Refs: nodejs/security-wg#973 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Mestery <mestery@protonmail.com>
Referencing: https://github.com/nodejs/node/actions/runs/5049673263/jobs/9059426826
The text was updated successfully, but these errors were encountered: