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

Avoid failing when receiving a 429 without a Retry-After header #4309

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bk2204
Copy link
Member

@bk2204 bk2204 commented Nov 17, 2020

No description provided.

Base automatically changed from master to main February 2, 2021 16:36
@xbbsky
Copy link

xbbsky commented Mar 2, 2022

Hi, we also met this crash when using git lfs. Would this fix be merged into 2.x version?

@bk2204
Copy link
Member Author

bk2204 commented Mar 2, 2022

Hey,

No, this hasn't been merged yet. We're waiting for someone to verify that this fix works for them. If you're willing to do so, then I'll finish writing it up and we can include it in the next release. When CI is finished building, it will produce binaries if you need one to test.

The 2.x series is now EOL and won't be seeing any updates.

@xbbsky
Copy link

xbbsky commented Mar 3, 2022

Hey,

No, this hasn't been merged yet. We're waiting for someone to verify that this fix works for them. If you're willing to do so, then I'll finish writing it up and we can include it in the next release. When CI is finished building, it will produce binaries if you need one to test.

The 2.x series is now EOL and won't be seeing any updates.

I'd like to test the binaries if you need. How could I contribute the test results?

@bk2204
Copy link
Member Author

bk2204 commented Mar 3, 2022

Just comment here if they solve the problem for you (or if they don't).

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