animegasan / luci-app-speedtest

LuCI support for speedtestcpp
Apache License 2.0
6 stars 2 forks source link

Feature request: Automatic periodic speedtests #3

Closed Cypher01 closed 1 week ago

Cypher01 commented 3 weeks ago

Great app, exactly what I was looking for, thx!

Can you add a functionality to automatically run periodic speedtests, e.g. every hour?

I already tried to add speedtestcpp to the scheduled tasks, but the results are not visible on LuCI, because you parse and write /etc/speedtest_result by yourself.

Alternatively, you could change your implementation to not parse a result and add the re-formatted line to /etc/speedtest_result, but instead write the raw JSON response of speedtestcpp to the file and parse it on demand for LuCI.

animegasan commented 3 weeks ago

Great app, exactly what I was looking for, thx!

Can you add a functionality to automatically run periodic speedtests, e.g. every hour?

I already tried to add speedtestcpp to the scheduled tasks, but the results are not visible on LuCI, because you parse and write /etc/speedtest_result by yourself.

Alternatively, you could change your implementation to not parse a result and add the re-formatted line to /etc/speedtest_result, but instead write the raw JSON response of speedtestcpp to the file and parse it on demand for LuCI.

The reason I save the results in the file "/etc/speedtest_result" is because it will make monitoring easier on the Results page.

If you want to add a schedule, I can add it in the next update.

Maybe I will create a settings page for the speedtest schedule. Any suggestions for how the settings page should be created?

BTW, Thanks for the ideas and suggestions!

animegasan commented 1 week ago

@Cypher01 image

Is it good if it is like this?

Because there is a schedule option, on the results page there will be an option to download or delete all data so that it can be examined by a third party. image

Cypher01 commented 1 week ago

Awesome! I will test it soon. Thx!