-
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
Set secure connection as default #800
base: master
Are you sure you want to change the base?
Conversation
Speedtest servers don't support HTTP connections anymore so I renamed the `--secure` parameter to `--insecure` and set it to False by default.
@sivel jfyi, because of this the default run no longer works: $ speedtest-cli
Retrieving speedtest.net configuration...
Cannot retrieve speedtest configuration
ERROR: HTTP Error 403: Forbidden |
Sorry about not mentioning it, but the reason why I submitted this PR is because (as @vaygr wrote), running |
Can confirm that this is now an issue from where I test in UK. |
Not sure why it lets me review this PR, but I would like to see these changes merged, so there... |
I don't think this works anymore 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)> |
Patch taken from sivel/speedtest-cli#800
Patch taken from sivel/speedtest-cli#800
Speedtest servers don't support HTTP connections anymore so I renamed the
--secure
parameter to--insecure
and set it to False by default.