Closed joezimjs closed 1 month ago
I've finally found the root of the issue, it looks like the format of the server-timing response has changed, and this is not accounted for in the module, universal-speediest, we use which is causing it to not process the upload data.
I'm going to look at fixing this this weekend, whether by patching that module or transitioning to the official Cloudflare module
Awesome. I was just looking at it today and noticed you were using a dependency to run the tests that hasn't been updated in 2 years... If I had the time I'd be jumping on this fix myself.
I looked at fixing this weekend, but was having issues building the speedtest library after implanting the fix, probably due to its age.
I also tried the Cloudflare library, but it doesn’t seem to play nicely with Node.
I can’t find an obvious better library, so I’m going to keep investigating the build issues
As a matter of interest, is Speedtest.net working correctly - just loops for me....
As a matter of interest, is Speedtest.net working correctly - just loops for me....
Nope. Never seems to finish a speed test.
Hi Bryce - any update on this please?
I understand the 3.4.0 stable release is imminent… presumably a fix for this won’t make it, unfortunately?
@ChrisSW58 I'm going to try to my best to get it patched, it's just been a busy couple of weeks.,
If you're able, can you download this test build and run it in Companion using the Developer Modules patch in the main window, and let me know if that build works for you?
I tried it and it works for me! Thanks @bryce-seifert
That has sorted the issue Bryce - many thanks and very well done. Just FYI, the SpeedTest.net option still loops and never ends, but I assume you have not looked at that code. Many thanks once again.
@bryce-seifert - just updated to Companion beta 7240 with your module version 1.0.3. SpeedTest starts and then stops immediately. Was working fine in the test build.
@ChrisSW58 What OS are you on? Are there any errors in the module log?
Also can you confirm cloudflare is still good in the build for you? It’s just Speedtest?
@bryce-seifert It's very weird - your test build is now timing out even if I go back beta builds. Test build on 5 sec timeouts. Am running on MacOS. Logs attached. 7240 1.0.3 module-log-30_08_2024-13_08_46.csv 7240 with test build module on SpeedTest-log-30_08_2024-13_06_00.csv
And this is only for Speedtest option? Cloudflare is still fine?
Here's the full log same for either service.
Just to verify, in the latest beta, are you running the “Run Speedtest” action after starting the module? I changed it from running automatically in order to try and avoid the “call timed out” error
I'm clearing the log file and then starting the connection from there. I reverted to your test module and did the same and it now seems to be OK - maybe I cocked it up before!
Here is the log from the 7240 having started the module the same way, and also the variables from it
Thanks for your prompt response by the way - much appreciated
In the latest beta (different from the test build), you will need to run the “Run Speedtest” action after starting the module. Perhaps I should change the “Stopped” variable to “Not Yet Run” or similar to indicate this behavior change
Yes!!! That's the issue...it's connected but you now need an explicit run SpeedTest command.... Sorry for my stupidity!! That is a big change - I didn't see it documented in the button help....You might get others confused by this....But seriously, very many thanks - I can sort that now.
No worries, yes I agree that I need to update the help documentation now that it’s functional.
Just curious, are both Speedtest and cloudflare services working for you now?
No - Speedtest looping as always on 5 sec timeout - here's the log
Got it , thank you for the log on that that is a helpful error. I will try and diagnose that next
@ChrisSW58 I believe speedtest.net tests should be fixed in the latest Companion beta.
Just updated to the latest beta and seems to be fine for me. Thanks for all of the hard work.
Yes! Both options working successfully. Many thanks for all your hard work
Companion v3.3.1 When I run a Cloudflare speed test, the upload speed variable never gets filled.