forked from haikuports/haikuports
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
speedtest_cli: use https by default.
Patch taken from sivel/speedtest-cli#800
- Loading branch information
Showing
2 changed files
with
59 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
From 74d662e1e00de05779c71717626bb7ec299bdf3c Mon Sep 17 00:00:00 2001 | ||
From: Strahinja Kustudic <[email protected]> | ||
Date: Tue, 1 Aug 2023 11:17:06 +0200 | ||
Subject: [PATCH] Set secure connection as default | ||
|
||
Speedtest servers don't support HTTP connections anymore so I renamed | ||
the `--secure` parameter to `--insecure` and set it to False by default. | ||
--- | ||
speedtest.py | 10 +++++----- | ||
1 file changed, 5 insertions(+), 5 deletions(-) | ||
|
||
diff --git a/speedtest.py b/speedtest.py | ||
index 186b52929..59540af66 100755 | ||
--- a/speedtest.py | ||
+++ b/speedtest.py | ||
@@ -946,7 +946,7 @@ class SpeedtestResults(object): | ||
""" | ||
|
||
def __init__(self, download=0, upload=0, ping=0, server=None, client=None, | ||
- opener=None, secure=False): | ||
+ opener=None, secure=True): | ||
self.download = download | ||
self.upload = upload | ||
self.ping = ping | ||
@@ -1087,7 +1087,7 @@ class Speedtest(object): | ||
"""Class for performing standard speedtest.net testing operations""" | ||
|
||
def __init__(self, config=None, source_address=None, timeout=10, | ||
- secure=False, shutdown_event=None): | ||
+ secure=True, shutdown_event=None): | ||
self.config = {} | ||
|
||
self._source_address = source_address | ||
@@ -1770,8 +1770,8 @@ def parse_args(): | ||
parser.add_argument('--source', help='Source IP address to bind to') | ||
parser.add_argument('--timeout', default=10, type=PARSER_TYPE_FLOAT, | ||
help='HTTP timeout in seconds. Default 10') | ||
- parser.add_argument('--secure', action='store_true', | ||
- help='Use HTTPS instead of HTTP when communicating ' | ||
+ parser.add_argument('--insecure', action='store_true', default=False, | ||
+ help='Use HTTP instead of HTTPS when communicating ' | ||
'with speedtest.net operated servers') | ||
parser.add_argument('--no-pre-allocate', dest='pre_allocate', | ||
action='store_const', default=True, const=False, | ||
@@ -1885,7 +1885,7 @@ def shell(): | ||
speedtest = Speedtest( | ||
source_address=args.source, | ||
timeout=args.timeout, | ||
- secure=args.secure | ||
+ secure=not args.insecure | ||
) | ||
except (ConfigRetrievalError,) + HTTP_ERRORS: | ||
printer('Cannot retrieve speedtest configuration', error=True) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters