Skip to content

Commit fa3217a

Browse files
committed
librespeed-cli: fix speedtest when server not available
If the server is down it can still be selected to test on it. This fixes the bug. See: - librespeed/speedtest-cli#94 Signed-off-by: Richard Muzik <richard.muzik@turris.com>
1 parent b9453f7 commit fa3217a

File tree

1 file changed

+25
-0
lines changed

1 file changed

+25
-0
lines changed
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
From b8fda15c50575ad7f1086fc8b0561b98e65c9c1e Mon Sep 17 00:00:00 2001
2+
From: user <user@linuxorg.org>
3+
Date: Wed, 18 Dec 2024 21:36:23 +0800
4+
Subject: [PATCH] fix start serverIdx in pingList when some servers are down
5+
6+
---
7+
speedtest/speedtest.go | 4 ++++
8+
1 file changed, 4 insertions(+)
9+
10+
diff --git a/speedtest/speedtest.go b/speedtest/speedtest.go
11+
index d461996..d8e49ad 100644
12+
--- a/speedtest/speedtest.go
13+
+++ b/speedtest/speedtest.go
14+
@@ -317,6 +317,10 @@ func SpeedTest(c *cli.Context) error {
15+
16+
// get the fastest server's index in the `servers` array
17+
var serverIdx int
18+
+ for serverIdx = range pingList {
19+
+ break
20+
+ }
21+
+
22+
for idx, ping := range pingList {
23+
if ping > 0 && ping <= pingList[serverIdx] {
24+
serverIdx = idx
25+

0 commit comments

Comments
 (0)