Compatibility with Homebridge v2

[jsiegenthaler]

0.1.19-beta.1

From jsiegenthaler:
Updated package.json to show compatibility with Homebridge v2
tested on homebdrige v2 with a motion sensor, all OK
Updated dependencies:
"http-auth": "^4.2.0",
"node-persist": "^2.1.0",
"request": "^2.88.2",
"selfsigned": "^2.4.1"
NOTE: a newer version of node-persist exists but all storage commands are async , requiring more code changes

[jsiegenthaler]

I'd like someone to review please

[benzman81]

This is awesome. No code changea at all needed?

[jsiegenthaler]

Nope. But note I only tested the motion sensor. Strictly speaking, we should test every sensor type as I see you have a chunk of code per sensor. If you had some non-conformal code, it would through an error in Homebridge v2. But of course, if the code is done well, and you stick to the defined characteristic values, then no error will occur.

[benzman81]

Ok, I will check how I can upgrade my docker homebridge to v2. This way some more accessories will be tested.

[jsiegenthaler]

I've created a full config with all supported accessories. Found this small issue, I'll fix it:

[24.09.2024, 06:30:57] Initializing platform accessory 'Fanv2-1'...
HAP-NodeJS WARNING: The accessory 'Fanv2-1' has an invalid 'Name' characteristic ('Fanv2-1'). Please use only alphanumeric, space, and apostrophe characters. Ensure it starts and ends with an alphabetic or numeric character, and avoid emojis. This may prevent the accessory from being added in the Home App or cause unresponsiveness.

Also fixed some names in the example config to be consistent with other examples

[jsiegenthaler]

Testing of each and ever sensor still needs to be done

[benzman81]

Awesome. As soon as I can update my homebridge to v2 I will test it and merge it.

[jsiegenthaler]

Noted one issue last night:
[24.09.2024, 22:57:33] [homebridge-http-webhooks] This plugin generated a warning from the characteristic 'Current Ambient Light Level': characteristic value expected valid finite number and received "NaN" (number). See https://homebridge.io/w/JtMGR for more info.
This will need fixing in the code

Caused by
[25.09.2024, 07:05:41] [homebridge-http-webhooks] Error: at CurrentAmbientLightLevel.characteristicWarning (C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2785:105) at CurrentAmbientLightLevel.validateUserInput (C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2673:14) at C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2284:24 at C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\util\once.ts:15:14 at HttpWebHookSensorAccessory.getState (C:\Users\jochen\Documents\GitHub\homebridge-http-webhooks\src\homekit\accessories\HttpWebHookSensorAccessory.js:155:5) at CurrentAmbientLightLevel.emit (node:events:519:28) at C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2267:14 at new Promise (<anonymous>) at CurrentAmbientLightLevel.handleGetRequest (C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2265:12) at CurrentAmbientLightLevel.toHAP (C:\Users\jochen\AppData\Roaming\npm\node_modules\homebridge\node_modules\hap-nodejs\src\lib\Characteristic.ts:2850:22)

I'll look at it tonight

[jsiegenthaler]

Saw a logging error and fixed it, and fixed some capitalisation errors in the logging in HttpWebHookThermostatAccessory.js, see commit log

[jsiegenthaler]

The issue "This plugin generated a warning from the characteristic 'Current Ambient Light Level'" is fixed. Root cause: the light sensor cannot have a minimum value of 0 for current light level. The minimum allowed is 0.0001. I added an IF statement to set state to 0.0001 when state is unknown

[jsiegenthaler]

@benzman81
I looked at some of the older PRs and if you want, we can incorporate some fixes into this PR. Example: #152 is a capitalisation inconsistency issue. I have an idea for a backwards compatible fix for this PR. I'd be happy to help. Let me know.

[benzman81]

Seems like the homebridge docker image is not yet available for v2. As soon as it is available I will integrate , test and merge.