-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
fix 403 err #788
base: master
Are you sure you want to change the base?
fix 403 err #788
Conversation
Thank you for the fix - LGTM! |
Why isn't this accepted yet? |
urls in get_servers like now returns 429 Error - Too Many Requests this prevents cli to retrieve servers lists and breaks the tests |
I doubt the usual link fix is the cause. Perhaps a bug in the site or code. I'll see what I can fix later. |
Using the flag --secure seems to work for me. |
Can you try my pull request? I doubt it will be different but worth a try. |
Thanks for your help @Loukious , yes I do use the modified version. I am on openmediavault ( up to date ), I also use piHole but there are no blocked requests, and if I disable it it doesn't change anything. I'm obviously not the only one with this concern: https://lafibre.info/tester-son-debit/speedtest-cli-error-unable-to-connect-to-servers-to-test-latency/ |
Hello all, Do you have solution ? Best Regards Anthony |
Same problem here... :-( |
can confirm that this pull request solves issue for me before after this pull request Retrieving speedtest.net configuration... |
i prefer #800 for the |
Fixing a link that was causing a 403 error