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 403 err #788

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix 403 err #788

wants to merge 1 commit into from

Conversation

SokDobry1
Copy link

Fixing a link that was causing a 403 error

@caboteria
Copy link

Thank you for the fix - LGTM!

@mmaridev
Copy link

Why isn't this accepted yet?

@maciej-kaznowski
Copy link

Works for me, thanks!

curl -s https://raw.githubusercontent.com/sivel/speedtest-cli/ef508f75e6a474968095913901fe119967b386e9/speedtest.py | python3 -

image

@EVOTk
Copy link

EVOTk commented Oct 16, 2022

Thanks, with your script I don't have the 403 error anymore! Unfortunately I have another error :(

image

image

@afl-iris
Copy link

urls in get_servers like
www.speedtest.net/speedtest-servers-static.php
or
www.speedtest.net/speedtest-servers.php

now returns 429 Error - Too Many Requests

this prevents cli to retrieve servers lists and breaks the tests

@SokDobry1
Copy link
Author

urls in get_servers like www.speedtest.net/speedtest-servers-static.php or www.speedtest.net/speedtest-servers.php

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.

@Loukious
Copy link

Using the flag --secure seems to work for me.

@EVOTk
Copy link

EVOTk commented Oct 18, 2022

It doesn't change anything for me :(
image

@Loukious
Copy link

Loukious commented Oct 18, 2022

It doesn't change anything for me :( image

Can you try my pull request? I doubt it will be different but worth a try.
curl -s https://raw.githubusercontent.com/sivel/speedtest-cli/b5df4bf507929d6014d4e7b13876c69fab9caf36/speedtest.py | python3
Edit: also are you sure you're using the modified version? Seems to me your using the master branch.

@EVOTk
Copy link

EVOTk commented Oct 18, 2022

Thanks for your help @Loukious , yes I do use the modified version.
But it doesn't work.

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.

image

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/

@ANTHONY-HESNAUX
Copy link

Hello all,
I confirm an error : Unable to connect to servers to test latency.

Do you have solution ?

Best Regards

Anthony

@EVOTk EVOTk mentioned this pull request Nov 21, 2022
@sgallou
Copy link

sgallou commented Nov 26, 2022

urls in get_servers like www.speedtest.net/speedtest-servers-static.php or www.speedtest.net/speedtest-servers.php

now returns 429 Error - Too Many Requests

this prevents cli to retrieve servers lists and breaks the tests

Same problem here... :-(

@Aivaras-s
Copy link

can confirm that this pull request solves issue for me

before
Retrieving speedtest.net configuration...
Cannot retrieve speedtest configuration
ERROR: HTTP Error 403: Forbidden

after this pull request

Retrieving speedtest.net configuration...
Testing from Telia Lietuva (xx.xx.xx.xx)...
Retrieving speedtest.net server list...
Selecting best server based on ping...

@milahu
Copy link

milahu commented Nov 26, 2023

i prefer #800 for the --insecure flag

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.