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

Rate limit exceeded #22

Open
jazzdan opened this issue Dec 6, 2023 · 1 comment
Open

Rate limit exceeded #22

jazzdan opened this issue Dec 6, 2023 · 1 comment

Comments

@jazzdan
Copy link

jazzdan commented Dec 6, 2023

I was doing development normally today, invoking ~10 builds/hour using buckle, and eventually Buckle broke:

$ buckle
Error: invalid type: map, expected a sequence at line 1 column 0

I looked in the releases.json and saw that it contained an error message:

$ cat /Users/dan/Library/Caches/buckle/releases.json
{"message":"API rate limit exceeded for a.b.c.d. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}

It seems like buckle should definitely not overwrite releases.json with an error message if it gets rate limited, and I'm surprised that I was rate limited at all given how lightly I was using it.

@benbrittain
Copy link
Owner

Yeah, that's no good at all. I haven't seen this occur personally but I'll definitely improve that logic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants