fwestenberg / reolink_dev

Home Assistant Reolink addon
MIT License
550 stars 101 forks source link

Add optional SMTP server for event based AI detection #453

Closed kmhallen closed 1 year ago

kmhallen commented 2 years ago

Webhooks provide event based motion detection, but not AI detection SMTP event for AI detection start, cleared on next periodic update

cpainchaud commented 2 years ago

Hi @kmhallen , This is a very interesting approach. One thing though I think it won't work with NVRs which can have only 1 SMTP server configured for all Cameras. In its state I would recommend NVR people to not use it

kmhallen commented 2 years ago

The unique SMTP port limitation is per Home Assistant integration. I couldn't figure out how to share variables between instances. The wording in the README could probably be updated to reflect that.

An NVR should work with a few changes because the email includes camera name and channel number.

I don't have an NVR to test with, only single cameras.