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

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
+671 −593

Conversation

Notmarrco
Copy link

This PR uses the one from @marksmayo

marksmayo and others added 2 commits January 23, 2023 17:03
@marksmayo
python 3.11 updates and fixes
d456ed6 
 - 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
@Notmarrco
fix 429 errors
7906c4b 
Use the new json servers list
bob-beck pushed a commit to openbsd/ports that referenced this pull request Feb 17, 2023
@landryb
benchmarks/speedtest-cli: unbreak by switching to a new upstream
bec7533 
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

@sylvaindd sylvaindd mentioned this pull request Feb 20, 2023
Closed
@mabed-fr
Copy link

mabed-fr commented Mar 2, 2023 •
edited
Loading

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
@landryb @ryanakca
benchmarks/speedtest-cli: unbreak by switching to a new upstream
455bc67 
listing ookla servers switched from a php url returning XML to a json API.
cf sivel/speedtest-cli#796
@jason-curtis jason-curtis mentioned this pull request Aug 24, 2023
Open
@jpph
Copy link

jpph commented May 4, 2024

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

@TheTumultuousUnicornOfDarkness
Copy link

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.

@FluffyDiscord FluffyDiscord mentioned this pull request Jun 27, 2024
Open
@imclint21
Copy link

Seems like this problem happen often in France

@NeckBeardPrince
Copy link

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.

Testing this I get

Retrieving speedtest.net configuration...
Cannot retrieve speedtest configuration
ERROR: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1006)>

@JonasZaoui2
Copy link

I got the same error since few days ..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@Notmarrco @sylvaindd @mabed-fr @marcthegreate @jpph @TheTumultuousUnicornOfDarkness @imclint21 @NeckBeardPrince @JonasZaoui2 @marksmayo