search

alexjustesen / speedtest-tracker

Speedtest Tracker is a self-hosted internet performance tracking application that runs speedtest checks against Ookla's Speedtest service.
https://speedtest-tracker.dev/
MIT License
2.8k stars 107 forks source link

Run a speedtest with ookla cause a 500 error #1744

Closed SiriosDev closed 2 weeks ago

SiriosDev commented 2 weeks ago

Describe the bug Run a speedtest with ookla cause a 500 error

To Reproduce Steps to reproduce the behavior:

  1. Open speed tracker
  2. Click on Run Speedtest
  3. Click on Ookla Speedtest

Environment (please complete the following information):

Screenshots image image

Logs

TypeError
App\Actions\Speedtests\RunOoklaSpeedtest::handle(): Argument #1 ($serverId) must be of type ?int, string given, called in /app/www/vendor/lorisleiva/laravel-actions/src/Concerns/AsObject.php on line 24
svenvg93 commented 2 weeks ago

How did you defined the server id's in the env variables?

They shoud be defined like below when using multiple servers. SPEEDTEST_SERVERS=3242,21509,61186

For a single server its like this SPEEDTEST_SERVERS=3242

SiriosDev commented 2 weeks ago

my bad I forgot to post the compose, I can't post it now, but this is certainly the fix, my brain skipped id divided by commas and so I entered domains divided by commas. I feel stupid for not noticing this after reading it so many times 🤦🤣

Thanks You Very Much

svenvg93 commented 2 weeks ago

No worries 😄 let us know if there are any problems after changing it to the ID's. Otherwise please close the issue if all works :)

SiriosDev commented 2 weeks ago

yep, worked thx 🙏