search

kukulich / home-assistant-jablotron100

Home Assistant custom component for JABLOTRON 100+ alarm system
MIT License
67 stars 24 forks source link

Added missing error log into check_serial_port method #31

Closed peterturza closed 2 years ago

peterturza commented 2 years ago

Added missing error log in the check_serial_port method. Spent few hours debugging with message "Jablotron is not available". I had to add this log to find out my /dev/hidraw0 was missing access for homeassistant system user.

kukulich commented 2 years ago

I think the logger level should be “debug”

peterturza commented 2 years ago

I was following the same pattern that is used on line #545, #596 and #726. The integration in this case doesn't work and unless you change your logging level setup for HA you never know why. That is why I used logger level "error"

kukulich commented 2 years ago

Ok.