Skip to content

Commit 9a75dfa

Browse files
[Bug] Fix server selection during manual test runs (#2176)
Co-authored-by: Alex Justesen <[email protected]>
1 parent f6f9cb9 commit 9a75dfa

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

app/Jobs/Ookla/SelectSpeedtestServerJob.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,9 @@ public function handle(): void
4343
return;
4444
}
4545

46-
// If preferred servers are set in the config, we can use that.
47-
if (! blank(config('speedtest.servers'))) {
46+
// If preferred servers are set in the config, we can use that,
47+
// but only if the test is scheduled.
48+
if ($this->result->scheduled && ! blank(config('speedtest.servers'))) {
4849
$this->updateServerId(
4950
result: $this->result,
5051
serverId: $this->getConfigServer(),

app/Livewire/Topbar/RunSpeedtestAction.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public function speedtestAction(): Action
3939
->form([
4040
Select::make('server_id')
4141
->label('Select Server')
42-
->helperText('Leave empty to run the speedtest without specifying a server.')
42+
->helperText('Leave empty to run the speedtest without specifying a server. Blocked servers will be skipped.')
4343
->options(function (): array {
4444
return array_filter([
4545
'Manual servers' => Ookla::getConfigServers(),

0 commit comments

Comments
 (0)