thevoltagesource / eagle-mqtt-bridge Goto Github PK

I don't have any log info to help to determine if this is a bug or not, but I want ask if this condition is handled:

If the MQTT broker/server goes offline for some time, will the bridge recover when the broker is restored?

If needed I can try to recreate the condition and capture any available log info.

TypeError: Cannot read properties of undefined (reading '0')
at processNode (/app/eagle-listener.js:101:31)
at processMessage (/app/eagle-listener.js:36:7)
at /app/eagle-listener.js:21:6
at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
at next (/app/node_modules/express/lib/router/route.js:137:13)
at Route.dispatch (/app/node_modules/express/lib/router/route.js:112:3)
at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
at /app/node_modules/express/lib/router/index.js:281:22
at Function.process_params (/app/node_modules/express/lib/router/index.js:341:12)
at next (/app/node_modules/express/lib/router/index.js:275:10)

Thank you for releasing this bridge process. It has been the perfect piece of glue logic for my legacy Eagle-200 due to intermittent lock up of the local API, as well as general problems with Rainforest's legacy Portal not being reliable.

I have solar panels that generate energy. I have not observed the "currentsummationdelivered" value decrease when the solar array is producing energy.

Would it be possible to add the "currentsummationreceived" value as well and update the docker container?

Currently the code dumps received messages out to the console. This is good for testing but in prod this is unneeded data leaking out to a log somewhere. Does NodeJS have a logger like Python? Should I just read an environment variable to see if I should output the log? Don't paralyze your self, start simple and learn more!

Is it possible to release this as a HACS addon? This would make install and management of the service more integrated with Home Assistant and would not require maintaining a separate management path for this Docker container.

Hi. Great work on this project!

wrt: Details of most Eagle messages are in the code and can be used to enable additional published data.

I'm trying to add in price and tier. I'm a noob so i'm likely missing some simple step.
Could you point me to some guidance on how to modify the correct files? Here is what I did in the listener file.
Thanks!

  case 'pricecluster':                             
     var price = msg.price[0]                         
     var tier = msg.tier[0]                              
     //{ devicemacid: [ '0xd8d5b90000003e58' ],                                                  
     //  metermacid: [ '0x00078100001d2c64' ],               
     //  timestamp: [ '0xffffffff' ],                                                                                     
     //  price: [ '0x00000048' ],                          
     //  currency: [ '0x0348' ],                                                                                        
     //  trailingdigits: [ '0x03' ],                
     //  tier: [ '0x01' ],                                
     //  starttime: [ '0xffffffff' ],                       
     //  duration: [ '0xffff' ],                                                                 
     //  ratelabel: [ 'Set by User' ],                       
     //  port: [ '/dev/ttySP0' ] }                                                                                        
     var message = {'meter/price': price, 'meter/price': tier}   

Error based on this change:

at processMessage (/app/eagle-listener.js:162:57)
at /app/eagle-listener.js:22:6
at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
at next (/app/node_modules/express/lib/router/route.js:137:13)
at Route.dispatch (/app/node_modules/express/lib/router/route.js:112:3)
at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
at /app/node_modules/express/lib/router/index.js:281:22
at Function.process_params (/app/node_modules/express/lib/router/index.js:335:12)
at next (/app/node_modules/express/lib/router/index.js:275:10)
at responseHandler (/app/node_modules/express-xml-bodyparser/lib/types/xml.js:64:9)

If SUMMATION_WATTS is not defined the app crashes with the following error.
TypeError: Cannot read property 'toUpperCase' of undefined

Need to address the undefined situation and make sure that all other variables work as intended if undefined or have bad data.

v1.1.0 won't start and gives this error message:

#
# Fatal error in , line 0
# unreachable code
#
#
#
#FailureMessage Object: 0x7ebac54c

I rolled back v1.0.0 and it works fine.