Skip to content
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

Handle API errors / rate limits more gracefully #555

Open
rajbos opened this issue Aug 28, 2023 · 0 comments
Open

Handle API errors / rate limits more gracefully #555

rajbos opened this issue Aug 28, 2023 · 0 comments
Labels
bug Something isn't working enhancement New feature or request Keep Exempt this from stalebot

Comments

@rajbos
Copy link
Contributor

rajbos commented Aug 28, 2023

The token I am using in a run has run into the rate limit for any API calls. In that case, the dependency-review-action get's an error back and reports:

Error: Dependency review is not supported on this repository. Please ensure that Dependency graph is enabled along with GitHub Advanced Security on private repositories, see https://github.com/devops-actions/load-available-actions/settings/security_analysis

This is misleading, as the action has been working fine before (it's a public repo in a normal user account) and no settings where changed. The token that is used is just out of it's rate limit bounds. Users are send in the wrong direction.

Expected:
Either wait for the ratelimit to reset, or just show the error from the API

I suspect this is the first API call that get's made. Adding more graceful error handling here will already help a lot.

@febuiles febuiles added bug Something isn't working enhancement New feature or request labels Sep 11, 2023
@febuiles febuiles added the Keep Exempt this from stalebot label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request Keep Exempt this from stalebot
Projects
None yet
Development

No branches or pull requests

2 participants