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

Fix 429 errors #796

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix 429 errors #796

wants to merge 2 commits into from

Conversation

Notmarrco
Copy link

This PR uses the one from @marksmayo

marksmayo and others added 2 commits January 23, 2023 17:03
 - cleaned up (object)s
 - fixed some docstrings
 - added trailing commas introduced since python 3.5, 3.6+
 - tidied strings
 - converted some strings to f-strings
Use the new json servers list
bob-beck pushed a commit to openbsd/ports that referenced this pull request Feb 17, 2023
listing ookla servers switched from a php url returning XML to a json API.
cf sivel/speedtest-cli#796
@sylvaindd
Copy link

Just tried it and works fine for me.
My HomeAssistant integration hasn't been working for quite a few months now because of this issue.

It would be really nice if this could be merged and released

@mabed-fr
Copy link

mabed-fr commented Mar 2, 2023

Hello,

Same 429 cli, http and curl

→ speedtest --list
Retrieving speedtest.net configuration...
→ speedtest
Retrieving speedtest.net configuration...
Testing from Orange (XXXXXXXXXXX)...
Retrieving speedtest.net server list...
Selecting best server based on ping...
ERROR: Unable to connect to servers to test latency.
~ → curl -iL https://www.speedtest.net/speedtest-servers.php
HTTP/2 429 
date: Thu, 02 Mar 2023 18:57:11 GMT
content-type: text/html
content-length: 591
retry-after: 0
accept-ranges: bytes
cf-cache-status: DYNAMIC
set-cookie: __cf_bm=lsVV04VP4EnXe_XACbglTNypxfwSwVdCSBZCsKmb5_A-1677783431-0-AfzPaNeYOsnL5DGl66fR1OEsrx7cZqQIPsosKJdrzyfOmli02FaYmmpSXco13lETXCmgU0VgfrY5Mk2mj/t5r+U=; path=/; expires=Thu, 02-Mar-23 19:27:11 GMT; domain=.www.speedtest.net; HttpOnly; Secure; SameSite=None
server: cloudflare
cf-ray: 7a1bdd2f09e40d73-MRS

<html><head><title>Too Many Requests</title></head><body><h1>429 Error - Too Many Requests</h1><p>Your IP address is making too many requests. We ask that you limit requests to no more than 1 per second.</p><p>We make an attempt to be flexible, allowing small bursts in traffic. You have exceeded this threshold however and have been blocked.</p><p>Please reduce the speed at which your requests are made and try again after a while. If you feel you have reached this message in error, please contact us at noc @ speedtest.net and reference this error code.</p><p>Thank you</p></body></html>

speedtest_broken

Can you merge ?

@marcthegreate
Copy link

marcthegreate commented Mar 3, 2023 via email

@mabed-fr
Copy link

mabed-fr commented Mar 3, 2023

Ok, I apologize, I was not aware I am very new at Github and also CLI stuff, so could I just use the CLI command networkquality instead? Brian Lauterbach - Sent using Spark Mail from Apple© MacBook Air M2 Sent with Spark

On Mar 2, 2023 at 12:58 PM -0600, Mathieu BEDOS @.>, wrote: Same 429 cli and http .. ~ → curl -iL https://www.speedtest.net/speedtest-servers.php HTTP/2 429 date: Thu, 02 Mar 2023 18:57:11 GMT content-type: text/html content-length: 591 retry-after: 0 accept-ranges: bytes cf-cache-status: DYNAMIC set-cookie: __cf_bm=lsVV04VP4EnXe_XACbglTNypxfwSwVdCSBZCsKmb5_A-1677783431-0-AfzPaNeYOsnL5DGl66fR1OEsrx7cZqQIPsosKJdrzyfOmli02FaYmmpSXco13lETXCmgU0VgfrY5Mk2mj/t5r+U=; path=/; expires=Thu, 02-Mar-23 19:27:11 GMT; domain=.www.speedtest.net; HttpOnly; Secure; SameSite=None server: cloudflare cf-ray: 7a1bdd2f09e40d73-MRS <title>Too Many Requests</title>

429 Error - Too Many Requests

Your IP address is making too many requests. We ask that you limit requests to no more than 1 per second.

We make an attempt to be flexible, allowing small bursts in traffic. You have exceeded this threshold however and have been blocked.

Please reduce the speed at which your requests are made and try again after a while. If you feel you have reached this message in error, please contact us at noc @ speedtest.net and reference this error code.

Thank you

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.>

Good morning,

That's not the topic but yes

ryanakca pushed a commit to ryanakca/OpenBSD-ports that referenced this pull request Mar 22, 2023
listing ookla servers switched from a php url returning XML to a json API.
cf sivel/speedtest-cli#796
@jpph
Copy link

jpph commented May 4, 2024

Can it be updated ?
Freebsd package is failing to retrive server list

@TheTumultuousUnicornOfDarkness

I tested your script by using curl -s https://raw.githubusercontent.com/Notmarrco/speedtest-cli/master/speedtest.py | python3 -: it works like a charm, thanks.
Hope this PR will be merged soon.

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