Having been messing around with some stuff I worked a way to self trigger some tests that I wanted to do on the HA <> MSR integration
This got me wondering if there's an entity that changes state / is exposed when a configured controller goes off line? I can't see one but thought it might be hidden or something?
Cheers
C
Using build 25328 and having the following users.yaml configuration:
users:
# This section defines your valid users.
admin: *******
groups:
# This section defines your user groups. Optionally, it defines application
# and API access restrictions (ACLs) for the group. Users may belong to
# more than one group. Again, no required or special groups here.
admin_group:
users:
- admin
applications: true # special form allows access to ALL applications
guests:
users: "*"
applications:
- dashboard
api_acls:
# This ACL allows users in the "admin" group to access the API
- url: "/api"
group: admin_group
allow: true
log: true
# This ACL allows anyone/thing to access the /api/v1/alive API endpoint
- url: "/api/v1/alive"
allow: true
session:
timeout: 7200 # (seconds)
rolling: true # activity extends timeout when true
# If log_acls is true, the selected ACL for every API access is logged.
log_acls: true
# If debug_acls is true, even more information about ACL selection is logged.
debug_acls: true
My goal is to allow anonymous user to dashboard, but MSR is still asking for a password when trying to access that. Nothing in the logs related to dashboard access. Probably an error in the configuration, but help needed to find that. Tried to put url: "/dashboard" under api_acls, but that was a long shot and didn't work.
I use Virtual Entity Controller virtual switches which I turn on via webhooks from other applications. Once a switch triggers and turns on, I can then activate associated rules.
I would like each virtual switch to automatically turn off after a configurable time (e.g., 5 seconds, 10 seconds).
Is there a better way to achieve this auto-off behavior instead of creating a separate rule for each switch that uses the 'Condition must be sustained for' option to turn it off?
With a large number of these switches (and the associated turn-off rules), I'm checking to see if there is a simpler approach.If not, could this be a feature request to add an auto-off timer directly to the virtual switches.
Thanks
Reactor (Multi-hub) latest-26011-c621bbc7
VirtualEntityController v25356
Synology Docker
TL;DR: Format of data in storage directory will soon change. Make sure you are backing up the contents of that directory in its entirety, and you preserve your backups for an extended period, particularly the backup you take right before upgrading to the build containing this change (date of that is still to be determined, but soon). The old data format will remain readable (so you'll be able to read your pre-change backups) for the foreseeable future.
In support of a number of other changes in the works, I have found it necessary to change the storage format for Reactor objects in storage at the physical level.
Until now, plain, standard JSON has been used to store the data (everything under the storage directory). This has served well, but has a few limitations, including no real support for native JavaScript objects like Date, Map, Set, and others. It also is unable to store data that contains "loops" — objects that reference themselves in some way.
I'm not sure exactly when, but in the not-too-distant future I will publish a build using the new data format. It will automatically convert existing JSON data to the new format. For the moment, it will save data in both the new format and the old JSON format, preferring the former when loading data from storage. I have been running my own home with this new format for several months, and have no issues with data loss or corruption.
A few other things to know:
If you are not already backing up your storage directory, you should be. At a minimum, back this directory up every time you make big changes to your Rules, Reactions, etc.
Your existing JSON-format backups will continue to be readable for the long-term (years). The code that loads data from these files looks for the new file format first (which will have a .dval suffix), and if not found, will happily read (and convert) a same-basenamed .json file (i.e. it looks for ruleid.dval first, and if it doesn't find it, it tries to load ruleid.json). I'll publish detailed instructions for restoring from old backups when the build is posted (it's easy).
The new .dval files are not directly human-readable or editable as easily as the old .json files. A new utility will be provided in the tools directory to convert .dval data to .json format, which you can then read or edit if you find that necessary. However, that may not work for all future data, as my intent is to make more native JavaScript objects directly storable, and many of those objects cannot be stored in JSON.
You may need to modify your backup tools/scripts to pick up the new files: if you explicitly name .json files (rather than just specifying the entire storage directory) in your backup configuration, you will need to add .dval files to get a complete, accurate backup. I don't think this will be an issue for any of you; I imagine that you're all just backing up the entire contents of storage regardless of format/name, that is the safest (and IMO most correct) way to go (if that's not what you're doing, consider changing your approach).
The current code stores the data in both the .dval form and the .json form to hedge against any real-world problems I don't encounter in my own use. Some future build will drop this redundancy (i.e. save only to .dval form). However, the read code for the .json form will remain in any case.
This applies only to persistent storage that Reactor creates and controls under the storage tree. All other JSON data files (e.g. device data for Controllers) are unaffected by this change and will remain in that form. YAML files are also unaffected by this change.
This thread is open for any questions or concerns.
Just another thought. Adding devices from my Home Assistant / Zigbee2MQTT integration. Works perfectly but they always add as their IEEE address. Some of these devices have up to 10 entities associated, and the moment they are renamed to something sensible, each of those entities 'ceases to exist' in MSR. I like things tidy, and deleting each defunct entity needs 3 clicks.
Any chance of a 'bulk delete' option?
No biggy as I've pretty much finished my Z-wave migration and I don't expect to be adding more than 2 new Zigbee devices
Cheers
C
Build 21228 has been released. Docker images available from DockerHub as usual, and bare-metal packages here.
Home Assistant up to version 2021.8.6 supported; the online version of the manual will now state the current supported versions;
Fix an error in OWMWeatherController that could cause it to stop updating;
Unify the approach to entity filtering on all hub interface classes (controllers); this works for device entities only; it may be extended to other entities later;
Improve error detail in messages for EzloController during auth phase;
Add isRuleSet() and isRuleEnabled() functions to expressions extensions;
Implement set action for lock and passage capabilities (makes them more easily scriptable in some cases);
Fix a place in the UI where 24-hour time was not being displayed.
I have tried numerous ways to define a recurring annual period, for example from December 15 to January 15. No matter which method I try - after and before, between, after and/not after, Reactor reports "waiting for invalid date, invalid date. Some constructs also seem to cause Reactor to hang, timeout and restart. For example "before January 15 is evaluated as true, but reports "waiting for invalid date, invalid date". Does anyone have a tried and true method to define a recurring annual period? I think the "between" that I used successfully in the past may have broken with one of the updates.
Good evening all,
For about the past week or so, I've been having problems with a specific rule in my home automation that controls when my home goes from an Away mode to Home mode. One of the conditions it checked for was my alarm panel, when it changed from Armed Away to Disarmed. There seems to have been a firmware update on the panel that added an intermittent step of "pending", and I can't say for certain it happens 100% of the time.
Is there a way to write a condition that so it changes from one condition, to the next, and then another condition? As in, Home alarm changes from armed_away to pending to disarmed.
Thanks.
No idea how easy this would be. During my migration away from Z-wave I've been replacing the Z-wave devices with Sonoff which has broken some of my automations.
Any chance of a 'Test Reaction' function to call out which ones are broken because an entity no longer exists? Without actually running the reaction?
Or does this exist already and I'm just not aware of how to do it? Obviously I can see entities that are no longer available, but not quite what I'm looking for.
I guess it's something of an edge case so no huge issue.
TIA!
C
I'm sure this has been asked, and answered, but damned if I can figure it out
Use case: I have a rear garden with lights. A door from the kitchen into the garden and a door from the garage.
Currently if I open the kitchen door the lights come on (yay) and a 3 minute delay starts.
After 3 minutes, no matter what else happens, the lights go off (Boo! But also yay!)
What I would like is for the 3 minute delay until the lights go off to start from the latest door open event.
That is, if I'm going from kitchen to garage, and back again, the lights stay on until there's three minutes of no activity.
I've tried 'hacking' with a virtual switch, but can't seem to stop the delay.
Any pointers?
TIA
C
MQTT interface... time for some testing. Where are my experts?
-
You need to make sure the response to that command actually includes the data you want. I've found that some Tasmota devices return incomplete state on this query. For example, I have a node set up with two relays, and this query does not provide the state of the two relays. The alternate choice, then, is to send
cmnd/%topic%Powerwith no payload, and the node replies with aRESULTtopic and the current state (which the template understands).My guess is this issue is moot, however, if you're using an MQTT broker that supports QoS and retain.
-
You need to make sure the response to that command actually includes the data you want. I've found that some Tasmota devices return incomplete state on this query. For example, I have a node set up with two relays, and this query does not provide the state of the two relays. The alternate choice, then, is to send
cmnd/%topic%Powerwith no payload, and the node replies with aRESULTtopic and the current state (which the template understands).My guess is this issue is moot, however, if you're using an MQTT broker that supports QoS and retain.
I'm really sorry but I'm failing getting this running - It seems to not be able to find the MQTT files on startup. Running in Docker and have created the correct directory structure with files. It can read my edited config yaml files from home/kevin/reactor/config OK. Permissions are OK.
Have a lot of MQTT experience but not so hot on Linux/Docker this was my run command
docker run --name reactor -d --restart on-failure -p 8111:8111 -v /home/kevin/reactor:/var/reactor --mount type=bind,src=/etc/localtime,target=/etc/localtime toggledbits/reactor:latest-generic-amd64I have
[latest-22004]2022-01-07T02:08:16.554Z <Structure:INFO> Structure#1 loading controller interface mqtt (MQTTController) [latest-22004]2022-01-07T02:08:16.560Z <MQTTController:null> Module MQTTController v21361 [latest-22004]2022-01-07T02:08:16.561Z <Controller:CRIT> Controller: failed to load mqtt implementation file:///var/reactor/ext/MQTTController/MQTTController.js: [Error]Error: Cannot find module 'mqtt' Require stack: - /var/reactor/ext/MQTTController/MQTTController.js [latest-22004]2022-01-07T02:08:16.562Z <Controller:CRIT> Error: Cannot find module 'mqtt' Require stack: - /var/reactor/ext/MQTTController/MQTTController.js Error: Cannot find module 'mqtt' Require stack: - /var/reactor/ext/MQTTController/MQTTController.js ... -
Did you just drop it in the folder, or did you go through all the installation steps? The message logged suggests that step 5 wasn't completed.
-
I followed the instructions. Including step 5. I have a feeling I saw an error at some stage in that install so I will revisit that step and report back.
This is what I see when running step5 ./install.sh
Installing dependencies... make: Entering directory '/home/kevin/reactor/ext/MQTTController/node_modules/in otify/build' CXX(target) Release/obj.target/inotify/src/bindings.o In file included from ../src/bindings.cc:2:0: ../src/bindings.h:10:27: error: ‘Handle’ has not been declared static void Initialize(Handle<Object> target); ^~~~~~ ../src/bindings.h:10:33: error: expected ‘,’ or ‘...’ before ‘<’ token static void Initialize(Handle<Object> target); ^ ../src/bindings.cc:11:27: error: variable or field ‘Initialize’ declared void void Inotify::Initialize(Handle<Object> exports) { ^~~~~~ ../src/bindings.cc:11:27: error: ‘Handle’ was not declared in this scope ../src/bindings.cc:11:27: note: suggested alternative: ‘rand_r’ void Inotify::Initialize(Handle<Object> exports) { ^~~~~~ rand_r ../src/bindings.cc:11:40: error: expected primary-expression before ‘>’ token void Inotify::Initialize(Handle<Object> exports) { ^ ../src/bindings.cc:11:42: error: ‘exports’ was not declared in this scope void Inotify::Initialize(Handle<Object> exports) { ^~~~~~~ ../src/bindings.cc: In static member function ‘static Nan::NAN_METHOD_RETURN_TYP E NodeInotify::Inotify::AddWatch(Nan::NAN_METHOD_ARGS_TYPE)’: ../src/bindings.cc:146:43: error: no matching function for call to ‘v8::Value::T oObject()’ Local<Object> args_ = info[0]->ToObject(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2672:44: note: candidate: v8::MaybeLocal<v8::Object> v8::Value::ToObject(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT MaybeLocal<Object> ToObject( ^~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2672:44: note: candidate expects 1 argument, 0 provided In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2686:31: note: candidate: v8::Local<v8::Object> v8::Value::ToObject(v8::Isolate*) const Local<Object> ToObject(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2686:31: note: candidate expects 1 argument, 0 provided Local<Object> ToObject(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:149:27: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(path_sym)) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:154:31: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(callback_sym) || ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:155:28: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] !args_->Get(callback_sym)->IsFunction()) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:160:32: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(watch_for_sym)) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:163:33: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] if (!args_->Get(watch_for_sym)->IsInt32()) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:166:36: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] mask |= args_->Get(watch_for_sym)->Int32Value(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:166:50: error: no matching function for call to ‘v8::Value::I nt32Value()’ mask |= args_->Get(watch_for_sym)->Int32Value(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate: v8::Maybe<int> v8::Value::Int32Value(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT Maybe<int32_t> Int32Value(Local<Context> context) const ; ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate expects 1 argument, 0 provided ../src/bindings.cc:172:45: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] String::Utf8Value path(args_->Get(path_sym)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:172:46: error: no matching function for call to ‘v8::String:: Utf8Value::Utf8Value(v8::Local<v8::Value>)’ String::Utf8Value path(args_->Get(path_sym)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3135:5: note: candidate: v 8::String::Utf8Value::Utf8Value(v8::Isolate*, v8::Local<v8::Value>) Utf8Value(Isolate* isolate, Local<v8::Value> obj); ^~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3135:5: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:182:61: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] inotify->handle()->Set(descriptor, args_->Get(callback_sym)); ^ -
This is what I see when running step5 ./install.sh
Installing dependencies... make: Entering directory '/home/kevin/reactor/ext/MQTTController/node_modules/in otify/build' CXX(target) Release/obj.target/inotify/src/bindings.o In file included from ../src/bindings.cc:2:0: ../src/bindings.h:10:27: error: ‘Handle’ has not been declared static void Initialize(Handle<Object> target); ^~~~~~ ../src/bindings.h:10:33: error: expected ‘,’ or ‘...’ before ‘<’ token static void Initialize(Handle<Object> target); ^ ../src/bindings.cc:11:27: error: variable or field ‘Initialize’ declared void void Inotify::Initialize(Handle<Object> exports) { ^~~~~~ ../src/bindings.cc:11:27: error: ‘Handle’ was not declared in this scope ../src/bindings.cc:11:27: note: suggested alternative: ‘rand_r’ void Inotify::Initialize(Handle<Object> exports) { ^~~~~~ rand_r ../src/bindings.cc:11:40: error: expected primary-expression before ‘>’ token void Inotify::Initialize(Handle<Object> exports) { ^ ../src/bindings.cc:11:42: error: ‘exports’ was not declared in this scope void Inotify::Initialize(Handle<Object> exports) { ^~~~~~~ ../src/bindings.cc: In static member function ‘static Nan::NAN_METHOD_RETURN_TYP E NodeInotify::Inotify::AddWatch(Nan::NAN_METHOD_ARGS_TYPE)’: ../src/bindings.cc:146:43: error: no matching function for call to ‘v8::Value::T oObject()’ Local<Object> args_ = info[0]->ToObject(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2672:44: note: candidate: v8::MaybeLocal<v8::Object> v8::Value::ToObject(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT MaybeLocal<Object> ToObject( ^~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2672:44: note: candidate expects 1 argument, 0 provided In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2686:31: note: candidate: v8::Local<v8::Object> v8::Value::ToObject(v8::Isolate*) const Local<Object> ToObject(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2686:31: note: candidate expects 1 argument, 0 provided Local<Object> ToObject(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:149:27: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(path_sym)) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:154:31: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(callback_sym) || ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:155:28: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] !args_->Get(callback_sym)->IsFunction()) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:160:32: error: no matching function for call to ‘v8::Object:: Has(v8::Local<v8::String>&)’ if (!args_->Has(watch_for_sym)) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, v8::Local<v8::Value>) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3590:37: note: candidate expects 2 arguments, 1 provided /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate: v8::Maybe<bool> v8::Object::Has(v8::Local<v8::Context>, uint32_t) V8_WARN_UNUSED_RESULT Maybe<bool> Has(Local<Context> context, uint32_t index) ; ^~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3596:37: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:163:33: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] if (!args_->Get(watch_for_sym)->IsInt32()) { ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:166:36: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] mask |= args_->Get(watch_for_sym)->Int32Value(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:166:50: error: no matching function for call to ‘v8::Value::I nt32Value()’ mask |= args_->Get(watch_for_sym)->Int32Value(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate: v8::Maybe<int> v8::Value::Int32Value(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT Maybe<int32_t> Int32Value(Local<Context> context) const ; ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate expects 1 argument, 0 provided ../src/bindings.cc:172:45: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] String::Utf8Value path(args_->Get(path_sym)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:172:46: error: no matching function for call to ‘v8::String:: Utf8Value::Utf8Value(v8::Local<v8::Value>)’ String::Utf8Value path(args_->Get(path_sym)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3135:5: note: candidate: v 8::String::Utf8Value::Utf8Value(v8::Isolate*, v8::Local<v8::Value>) Utf8Value(Isolate* isolate, Local<v8::Value> obj); ^~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3135:5: note: candidate expects 2 arguments, 1 provided ../src/bindings.cc:182:61: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] inotify->handle()->Set(descriptor, args_->Get(callback_sym)); ^IN two posts because was too long - sorry
In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:182:62: warning: ‘bool v8::Object::Set(v8::Local<v8::Value>, v8::Local<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declaratio ns] inotify->handle()->Set(descriptor, args_->Get(callback_sym)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3499:22: note: declared he re bool Set(Local<Value> key, Local<Value> value)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc: In static member function ‘static Nan::NAN_METHOD_RETURN_TYP E NodeInotify::Inotify::RemoveWatch(Nan::NAN_METHOD_ARGS_TYPE)’: ../src/bindings.cc:195:31: error: no matching function for call to ‘v8::Value::I nt32Value()’ watch = info[0]->Int32Value(); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate: v8::Maybe<int> v8::Value::Int32Value(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT Maybe<int32_t> Int32Value(Local<Context> context) const ; ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2709:40: note: candidate expects 1 argument, 0 provided ../src/bindings.cc: In static member function ‘static void NodeInotify::Inotify: :Callback(uv_poll_t*, int, int)’: ../src/bindings.cc:262:86: warning: ‘bool v8::Object::Set(v8::Local<v8::Value>, v8::Local<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declaratio ns] >Set(Nan::New<String>("watch").ToLocalChecked(), Nan::New<Integer>(event->wd)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3499:22: note: declared he re bool Set(Local<Value> key, Local<Value> value)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:263:87: warning: ‘bool v8::Object::Set(v8::Local<v8::Value>, v8::Local<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declaratio ns] Set(Nan::New<String>("mask").ToLocalChecked(), Nan::New<Integer>(event->mask)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3499:22: note: declared he re bool Set(Local<Value> key, Local<Value> value)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:264:91: warning: ‘bool v8::Object::Set(v8::Local<v8::Value>, v8::Local<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declaratio ns] Nan::New<String>("cookie").ToLocalChecked(), Nan::New<Integer>(event->cookie)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3499:22: note: declared he re bool Set(Local<Value> key, Local<Value> value)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:267:104: warning: ‘bool v8::Object::Set(v8::Local<v8::Value>, v8::Local<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarati ons] ing>("name").ToLocalChecked(), Nan::New<String>(event->name).ToLocalChecked()); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3499:22: note: declared he re bool Set(Local<Value> key, Local<Value> value)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:275:66: warning: ‘v8::Local<v8::Value> v8::Object::Get(v8::Lo cal<v8::Value>)’ is deprecated: Use maybe version [-Wdeprecated-declarations] Local<Value> value = handle->Get(Nan::New<Integer>(event->wd)); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:3553:51: note: declared he re V8_DEPRECATED("Use maybe version", Local<Value> Get(Local<Value> key)); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ ../src/bindings.cc:279:34: warning: ‘v8::Local<v8::Value> Nan::Callback::Call(v8 ::Local<v8::Object>, int, v8::Local<v8::Value>*) const’ is deprecated [-Wdepreca ted-declarations] callback.Call(handle, 1, argv); ^ In file included from ../src/node_inotify.h:14:0, from ../src/bindings.h:4, from ../src/bindings.cc:2: ../../nan/nan.h:1722:3: note: declared here Call(v8::Local<v8::Object> target ^~~~ ../src/bindings.cc:285:34: error: no matching function for call to ‘v8::Value::T oString()’ handle->Delete(wd->ToString()); ^ In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:67 :0, from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2668:44: note: candidate: v8::MaybeLocal<v8::String> v8::Value::ToString(v8::Local<v8::Context>) const V8_WARN_UNUSED_RESULT MaybeLocal<String> ToString( ^~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2668:44: note: candidate expects 1 argument, 0 provided In file included from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8-intern al.h:14:0, from /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:27, from /home/kevin/.cache/node-gyp/12.20.0/include/node/node.h:6 , from ../src/node_inotify.h:5, from ../src/bindings.h:4, from ../src/bindings.cc:2: /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2684:31: note: candidate: v8::Local<v8::String> v8::Value::ToString(v8::Isolate*) const Local<String> ToString(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8.h:2684:31: note: candidate expects 1 argument, 0 provided Local<String> ToString(Isolate* isolate) const); ^ /home/kevin/.cache/node-gyp/12.20.0/include/node/v8config.h:328:3: note: in defi nition of macro ‘V8_DEPRECATED’ declarator __attribute__((deprecated(message))) ^~~~~~~~~~ inotify.target.mk:112: recipe for target 'Release/obj.target/inotify/src/binding s.o' failed make: *** [Release/obj.target/inotify/src/bindings.o] Error 1 make: Leaving directory '/home/kevin/reactor/ext/MQTTController/node_modules/ino tify/build' kevin@NUC50:~/reactor/ext/MQTTController$ -
Seems like something is broken or out of date in your nodejs installation. How did you install node? And what version is it (run
node -v) -
Yeah, that's going to be a problem. Version 16.13 is the current, I believe, and would be a good choice.
-
Thankyou - that's worked. I had assumed the appropriate version of node was part of the Docker container.
So pleased to have found this as I use HA, MQTT, Hubitat extensively plus OpenHAB and Homey to a lesser extent so glad you are intending to support those two eventually.
-
It is part of the container, but you are not running in the container. That is a possibility, though, and I tried it, it seems to work, so an alternative to installing (or upgrading) nodejs outside of the container seems to be:
docker exec -it reactor /bin/sh cd /var/reactor/ext/MQTTController ./install.sh exitI'll add this to the installation docs for MQTTController.
-
It is part of the container, but you are not running in the container. That is a possibility, though, and I tried it, it seems to work, so an alternative to installing (or upgrading) nodejs outside of the container seems to be:
docker exec -it reactor /bin/sh cd /var/reactor/ext/MQTTController ./install.sh exitI'll add this to the installation docs for MQTTController.
All working well. However in the topic structure for echoed devices (from Hubitat anyway) the <entity-id> is used. Is it possible to instead or even better as well as, include the entity-name ?
reactor/<mqtt-id>/<ctrl-id>/<entity-id>/state/<capability-name>perhaps
reactor/<mqtt-id>/<ctrl-id>/<entity-name>/<entity-id>/state/<capability-name> -
The entity name can be changed or duplicated and is not an appropriate unique identifier for a device.
-
-
Topic hierarchy segments don't have to be unique - in fact I would say most aren't. Keeping the <entity-id> then within the hierarchy would ensure the topic remains unique. Having no easily human identifiable topic is not helpful to other integrations.
@xappo OK. And what about receiving messages for inbound action requests? Same
entity-name/entity-idconstruction? If you know the correct entity ID to form that message, what value is the name? If you don't know the ID and only use the name, how do you specify that, and what if the name isn't unique then? And what if the name and ID conflict, or an entity can be found for one but not the other (i.e. they don't match)? I'm not opposed to what you're proposing, but I want you to help me think through the consequences of it.@xappo said in MQTT interface... time for some testing. Where are my experts?:
Having no easily human identifiable topic is not helpful to other integrations.
I think that's a bit of a reach. The whole MQTT environment is so unstructured and freeform, with so many inconsistencies between players, that things like unique IDs (or MAC addresses or other cryptic and "hard to read" identifiers) are a regular occurrence, and anybody who uses MQTT a lot is used to it.
-
The incoming action request message to a topic must be complete with all segments as you can't wildcard a publish command. So the <entity-id> will always be unique and present there. You could should you wish subscribe to the topic from your code ignoring any <entity-name> segment with a + wildcard as you don't process it. So that should work fine.
Human readability.. I agree it's a bit of a reach but my whole system is MQTT based with over 10K topics and 1K+ devices - remembering what they all are and linking to or from those topics from other controllers requires some hints as to what those devices are. Later examination of logic requires understanding what devices are related. Having to refer back to Reactor config each time to identify a device isn't practical (or user friendly).
-
Good stuff. Let me see what I can cook up.
-
OK. Version 22008 of MQTTController is up on the downloads site (under extras).
New configuration key that you can put in the
echosection: setentity_identifiertocombinedand the telemetry topics will include the entity name; it adds another level to the topic as you suggested. This has no effect on received messages for entity actions; those still require the ID (but it's a secret/undocumented feature--you can give it the name instead of an ID and it will work if the name is unique), and the structure of the topic is the same (i.e. same number of levels, not changed). Online docs are updated as well. -
This is working well - thank you for the prompt update.
For me having the <entity-name> before <entity-id> in the topic hierarchy would have been more informative as you can then see a list of names rather than numbers in expanded topics with say MQTT explorer rather than having to drill down in each one, but I totally understand why it is so much easier in terms of your existing implementation to position it afterwards and this is very workable too.
Thank you again.
Kevin -
No worries. Version 22009 now. You can now set
entity_identifiertoid/name(as a synonym forcombined) orname/id. I'll fix up the documentation later.
MQTTController - Reactor - Multi-Hub Automation