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

Respect HTTP 429 and Retry-After header #628

Merged

Conversation

craigmiskell-gitlab
Copy link
Contributor

When a 429 is received, sleep for the length of time indicated by Retry-After, then try again.

Try up to 3 times before giving up, although the count can be configured with :ratelimit_retries if you want more or less

If no Retry-After is sent by the server, defaults to 2 seconds (something is better than nothing). That should only happen with quite old and unsupported versions of GitLab

When a 429 is received, sleep for the length of time indicated by Retry-After, then try again.

Try up to 3 times before giving up, although the count can be configured with :ratelimit_retries if you want more or less

If no Retry-After is sent by the server, defaults to 2 seconds (something is better than nothing). That should only happen with quite old and unsupported versions of GitLab
@NARKOZ NARKOZ merged commit 1e8c635 into NARKOZ:master Oct 11, 2021
@NARKOZ
Copy link
Owner

NARKOZ commented Oct 11, 2021

Thank you ❤️

@craigmiskell-gitlab
Copy link
Contributor Author

@NARKOZ You're welcome. I'm keen to use this addition; do you have any plans for a new version release anytime soon?

@NARKOZ
Copy link
Owner

NARKOZ commented Oct 12, 2021

I plan to release this week.

@craigmiskell-gitlab
Copy link
Contributor Author

I plan to release this week.

@NARKOZ It's been a while; any chance we could get that release soon? I would be very grateful.

@NARKOZ
Copy link
Owner

NARKOZ commented Dec 26, 2021

@craigmiskell-gitlab I've just released a new version and pushed to Rubygems https://github.com/NARKOZ/gitlab/releases/tag/v4.18.0

Sorry for the wait!

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

Successfully merging this pull request may close these issues.

None yet

2 participants