Fix: Skip SkipSpeedtestJob when env var is empty (alexjustesen#2626)

svenvg93 · alexjustesen · web-flow · commit d0377e15a711 · 2026-01-03T08:14:31.000-05:00
Co-authored-by: Alex Justesen &lt;alexjustesen@users.noreply.github.com&gt;
diff --git a/app/Jobs/Ookla/SkipSpeedtestJob.php b/app/Jobs/Ookla/SkipSpeedtestJob.php
@@ -40,9 +40,9 @@ public function middleware(): array
     public function handle(): void
     {
         /**
-         * Only skip IPs for scheduled tests.
+         * Skip if test is not scheduled or no IPs are configured to skip.
          */
-        if ($this->result->scheduled === false) {
+        if ($this->result->scheduled === false || empty(config('speedtest.preflight.skip_ips'))) {
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -40,9 +40,9 @@ public function middleware(): array`
`40`	`40`	`public function handle(): void`
`41`	`41`	`{`
`42`	`42`	`/**`
`43`		`- * Only skip IPs for scheduled tests.`
	`43`	`+ * Skip if test is not scheduled or no IPs are configured to skip.`
`44`	`44`	`*/`
`45`		`- if ($this->result->scheduled === false) {`
	`45`	`+ if ($this->result->scheduled === false \|\| empty(config('speedtest.preflight.skip_ips'))) {`
`46`	`46`	`return;`
`47`	`47`	`}`
`48`	`48`