Firmware updates

[georgeinva2004]

The Zwave Firmware Update project implies that all ZwaveJS projects already have an API key. Does that apply to ZwaveJS.Net? Does the PSI file embed that key somewhere so we don't have to request one for ourselves?

[marcus-j-davies]

Hi @georgeinva2004,

Whilst the projects are all under the ZWaveJS ORG on GitHub - the projects are owned by their respective developer as individuals - we don't get perks 😒

I founded the Node RED plugin, and of course the NET lib. I acquired a license for my Node RED project - as it's open source, so was free, but i allow users who use it in a corporate setting to enter a key (to override the embedded one)

The .NET lib will take on a similar approach (thank you for reminding me, I need to ask for one).

You can request one with no fuss as it's a home project via [email protected]

But one will be embedded before release, for non commercial-use

[marcus-j-davies]

I should note:

Once the non-commercial key is embedded, the requirement for a key will become optional when using the method.

[georgeinva2004]

I figured it would be something like that. Thanks for letting us know!

[marcus-j-davies]

I now have my key - I'll get it implemented soon 👍

[georgeinva2004]

Working on implementing the Firmware update functionality. I was able to secure a key. The NET Lib is throwing an exception in Controller.GetAvailableFirmwareUpdates(..). It looks like NewtonSoft is having a problem deserializing the result.updates token when there are no updates. Exception details below. Assuming that the return value is an array, I made the following change to line that deserializes the result.updates token:

List<FirmwareUpdateInfo> FUI = JsonConvert.DeserializeObject<List<FirmwareUpdateInfo>>(JO.SelectToken("result.updates").ToString());

As you can see, I just made the result a List and it appears to be working. Unfortunately, I don't have any devices that are reporting as needing a firmware update, so I can't fully test the functionality. I hope this helps...

Newtonsoft.Json.JsonSerializationException: 'Cannot deserialize the current JSON array (e.g. [1,2,3]) into type 'ZWaveJS.NET.FirmwareUpdateInfo' because the type requires a JSON object (e.g. {"name":"value"}) to deserialize correctly.
To fix this error either change the JSON to a JSON object (e.g. {"name":"value"}) or change the deserialized type to an array or a type that implements a collection interface (e.g. ICollection, IList) like List that can be deserialized from a JSON array. JsonArrayAttribute can also be added to the type to force it to deserialize from a JSON array.
Path '', line 1, position 1.'

This exception was originally thrown at this call stack:
[External Code]
ZWaveJS.NET.Controller.GetAvailableFirmwareUpdates.AnonymousMethod__0(Newtonsoft.Json.Linq.JObject) in Controller.cs
ZWaveJS.NET.Driver.WebsocketClient_MessageReceived(object, Websocket.Client.ResponseMessage) in Driver.cs

[marcus-j-davies]

Thanks @georgeinva2004 I'll get it patched over the weekend (when I implement the embedded key) 👍