banban525 / echonetlite2mqtt

ECHONET Lite to MQTT bridge.
MIT License
48 stars 7 forks source link

2.3.0→3.0.0で以下のようになって落ちてしまいます。 #28

Closed grandevice closed 11 months ago

grandevice commented 11 months ago

info [MQTT] prop changed: 寝室のエアコン operationStatus true info [MQTT] prop changed: 寝室のエアコン operationMode heating info [MQTT] prop changed: 寝室のエアコン targetTemperature 20 warn [ECHONETLite][raw] error send command: timeout 192.168.11.253 05ff01 013002 62 80 error: uncaughtException: Cannot read properties of undefined (reading 'length') TypeError: Cannot read properties of undefined (reading 'length') at EchoNetPropertyConverter.EchoNetLiteDataToNumber (/home/grandevice/echonetlite2mqtt/server/EchoNetPropertyConverter.ts:908:18) at EchoNetPropertyConverter.toObject (/home/grandevice/echonetlite2mqtt/server/EchoNetPropertyConverter.ts:663:23) at EchoNetPropertyConverter.toObject (/home/grandevice/echonetlite2mqtt/server/EchoNetPropertyConverter.ts:702:31) at EchoNetDeviceConverter.convertPropertyValue (/home/grandevice/echonetlite2mqtt/server/EchoNetDeviceConverter.ts:508:49) at EchoNetLiteController.setDevicePropertyPrivate (/home/grandevice/echonetlite2mqtt/server/EchoNetLiteController.ts:268:44) at async EchoNetLiteController.setDeviceProperty (/home/grandevice/echonetlite2mqtt/server/EchoNetLiteController.ts:209:5) at async /home/grandevice/echonetlite2mqtt/server/index.ts:561:3 at async MqttController.firePropertyChnagedEvent (/home/grandevice/echonetlite2mqtt/server/MqttController.ts:189:7) at async MqttClient. (/home/grandevice/echonetlite2mqtt/server/MqttController.ts:86:11)

echonetlite2mqtt_logs_internalsStatus (5).zip

banban525 commented 11 months ago

@grandevice

報告ありがとうございます。 エアコンの設定温度を変えるコマンドの成功後、設定温度をエアコンから取得していますが、そこで応答がなかったようです。 ただ、その時のデータは空のはずですが、なぜか受信データを解析してしまって、クラッシュしていると思われます。 少なくともクラッシュするのはバグなので、2~3日中に修正します。

ただ、根本原因の解明までは状況を再現できないので、難しいかもしれません。

banban525 commented 11 months ago

@grandevice 今回の原因でクラッシュしないように修正しました。 おそらくnode.jsで実行されていると思いますので、最新に更新してください。

ただ、こちらでは再現できなかったので根本原因までは不明です。 たぶんですが、直前のコマンドに別の通信が紛れ込んだ結果、誤動作したのではないかと思うのですが。

grandevice commented 11 months ago

@banban525 早速のご対応ありがとうございます。今のところクラッシュせずに稼働しております。 ありがとうございました。

banban525 commented 11 months ago

コメントありがとうございます。 直ったようで良かったです。このIssueはクローズしますね。