-
-
Notifications
You must be signed in to change notification settings - Fork 213
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cannot perform OTA updates from zwave-js-ui from packaged application #3411
Comments
I sincerly never saw such error. I need to ask @AlCalzone what that means |
@marcus-j-davies you had that in your dotnet wrapper, didn't you? |
Yup! Its the Its the FWIW (
|
I am seeing the same on my standalone install (no docker, just the exe as a service with nssm on windows). You shouldn't need any special configuration or setup to get it working, It just worked for me previously. |
@john159753 I agree with you but it's not that obvious, in order to package application I use a tool named pkg that does many things under the hoods that can create such issues that you don't see when running it without (like with docker, snap or nodejs). This time the problem is related to the driver that starting from version 12 is using some ESM package that is not fully supported by pkg... I will try to see if I find a way to make it work... the alternative is to fix this on driver side someway like switching to a cjs version of |
Should this be converted to a bug then? I understand it might take you a bit to fix, but it's preventing me from doing fw updates on my nodes. Thank you for all your effort though, it's appreciated! |
@garbled1 Sure, done. A quick fix would be to simply try another deploy method like docker, snap or manually built. |
@garbled1 Would the new npm deploy method be an option for you? |
I can take a look. My setup is a little complex, as I basically run it on a tiny VM, need to see how much needs to be installed to get npm running. npm is at least a better option for me than docker. How can I convert my settings from the binary version to that? It seems to reference different files? (specifically the ZWAVEJS_EXTERNAL_CONFIG option) |
Actually what you need to do is only to move everything you have in store directory to the new one or set as store path the store directory you are using now. Let's say you have your executable in folder |
@AlCalzone about this, could you switch to got-cjs? Otherwise i could do a try to create the pkg file against the new introduced esbuild bundle that should work |
Shouldn't this also be solved by esbuild bundling like Marcus wrote? |
Yeah that's the alternative, the only problem with that is I would add another layer of "abstraction" to the pkg version |
This should work in next version. In the meanwhile can anyone give a test to this? Linux x64: (This link will be valid until 12:00 CET of tomorrow 19/12/2023) |
Whenever I click on CHECK UPDATES or the OTA tab on a device, I get the following message:
Error while calling getAvailableFirmwareUpdates: Cannot check for firmware updates for node 11: Invalid host defined options (ZW0261)
I'm running 9.3.1 from zwave-js-ui-v9.3.1-linux.zip in the releases. Is there something I'm supposed to configure to get this feature working?
The text was updated successfully, but these errors were encountered: