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.

Sign up for GitHub

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

Jump to bottom

Retry when speedtest fails. #2657

Merged
merged 1 commit into from
Nov 4, 2024
Merged

Retry when speedtest fails. #2657

merged 1 commit into from
Nov 4, 2024

Conversation

suica-me
Copy link
Contributor

@suica-me suica-me commented Aug 29, 2024 •
edited
Loading

PR Description:

Retry the speedtest when it fails due to network fluctuations.

except (http.client.IncompleteRead, ConnectionResetError) as error:

Tests and Checks

  • I have tested the code!

@suica-me suica-me marked this pull request as ready for review September 1, 2024 05:05
@suica-me suica-me requested a review from Torxed as a code owner September 1, 2024 05:05
@Torxed Torxed merged commit e312367 into archlinux:master Nov 4, 2024
6 checks passed
@Torxed
Copy link
Member

Torxed commented Nov 4, 2024

Apologies for the delay in getting back to you on this one.
I finally had some time to sit down and test this, and it appears to work just as intended. I went ahead and merged it in prep for a new release.

@suica-me
Copy link
Contributor Author

suica-me commented Nov 4, 2024

Thank you so much for the comfirm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Torxed Torxed Torxed approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@suica-me @Torxed