Undesirable reboots when no LMS is present

[pmannk]

Hi, can you please look at adding a flag (even if it's something that has to be added manually via the nvs editor) that prevents reboots when no LMS is detected? (https://github.com/sle118/squeezelite-esp32/commit/ba156586e9fc55c9a8bc943312eee8bd8c65d239) seemed to introduce a behaviour where the device will reboot when LMS fails. For those of us who only use our devices for Bluetooth/Airplay/Spotify (and don't actually have an LMS server) it's undesirable to reboot when LMS fails to connect. It's an expected condition in our circumstances.

Thanks, I really appreciate this awesome project!

[sle118]

I'll let @philippe44 take a stab at this request, as this is in his area of expertise.

[philippe44]

This one is unfortunate as the only reason why I reboot is because WiFi would stop working properly. So squeezelite is stuck in a forever search loop for a server. We can add a flag, but IMHO, it would be better to figure out why WiFi stops working. It happens when you loose coverage for some time (like an AP reboot that takes a minute or two).

And this is also a LMS-centered project. Using it 100% without LMS is kind of defeating the purpose. Other esp32 projects like euphonium probably do a better job then

[pr10dr]

+1 For adding a flag to make the reboot optional. Additional the reboot could be indicated in the web ui logs in a more understandable way. Iam a new user focusing on the spotify streaming experience and the regular reboots and web ui showing "timeout" where not helping me in understanding that i might need an LMS (which i dont have any intentions to use) to get rid of the issue. Downgraded for now.

sidenote: Overall the documentation and user experience where GREAT (heart)

[philippe44]

Maybe I can add one option to squeezelite command line that disables the whole slimproto in which case it's more consistent with a usage without LMS. Having said that, you still have other projects that are more suitable

[philippe44]

When LMS server to be bound to is "-disable", LMS search will not be done and no reboot will happen. Please test and tell me if this works for you, I don't have time to try for myself. Remember that as this is a squeezelite-centered project, for example the audio backend is inside squeezelite so Airplay, BT and Spotify are nasty hacks I've added on top of it. In a proper architecture, audio backend should be an independent component that any client requestswhen needed. So again, better use projects like euphonium.

[pmannk]

Thanks for taking a look at this @philippe44. I appreciate it's a squeezelite-centered project, but it's so good at everything else too it would be a shame to alienate potential users with a hard LMS requirement.

I tested out your fix with 2.1347, I saw you had merged https://github.com/sle118/squeezelite-esp32/commit/1d65bf910623c330ab7e2da5b55d67971ef884cb. Thanks heaps! I tested on two devices today and both have been working flawlessly. No reboots to report, and I saw the log event as expected on boot: [00:00:01.469] squeezelite_main:816 LMS is disabled E (2945) messaging: [00:00:01.469] squeezelite_main:816 LMS is disabled

Entry from the NVS editor for anyone else who stumbles across this thread: autoexec1: squeezelite -b 500:2000 -d all=info -C 30 -W -n "Workshop" -o I2S -s -disable

[philippe44]

You should see that log every hour. I'm glad you like our little project, but a fair bit of features are missing when you don't use LMS. Here is a link to euphonium. One on the developer is the author of cspot and a really nice guy.