speedtest-cli
speedtest-cli copied to clipboard
Remove break in `get_servers`
Remove the break in the get_servers
method so that we can get the list of servers from all urls.
Thanks for submitting this. I was going to look for this exact problem. It resolves a key issue for me of not getting local area servers in the returned list.
Hello @sivel , any chance we can get this merged please.
I just noticed this issue too, I was wondering why speedtest-cli was not giving as good server alternatives as in other clients and found the break, should have looked at the pull requests first lol. Hopefully can be merged.
Just ran into this myself, was confused why two different systems and networks physically separated by only about a kilometer not only had very different lists but they also would refuse to run against server IDs not in their lists (which was making meaningful comparisons tricky!).
This line removal seems to entirely solve that, with many more servers now listed and reachable, although I note that the list shown via --list
now has most (though fascinatingly not all) entries duplicated.