Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reolink integration does not trigger binary_sensor.visitor in case video doorbell button pressed #91041

Closed
fr33pr0d opened this issue Apr 7, 2023 · 10 comments

Comments

@fr33pr0d
Copy link

fr33pr0d commented Apr 7, 2023

The problem

Reolink integration does not trigger binary_sensor.visitor in case video doorbell button pressed.

What version of Home Assistant Core has the issue?

core-2023.4.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Reolink IP NVR/camera

Link to integration documentation on our website

https://www.home-assistant.io/integrations/reolink

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

Unfortunately this issue persists and it looks like it made it worse than before. Now it detects about 20% of doorbell presses and even when it works, it "detects" it a good few minutes after the button press. Everything else works absolutely fine. I still have HTTP set, I also tried with it off and it doesn't work at all. Enabled it back and it works 2 times out of 10-ish.
Any help would be appreciated.

@home-assistant
Copy link

home-assistant bot commented Apr 7, 2023

Hey there @starkillerOG, mind taking a look at this issue as it has been labeled with an integration (reolink) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of reolink can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign reolink Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


reolink documentation
reolink source
(message by IssueLinks)

@starkillerOG
Copy link
Contributor

@fr33pr0d do you see any repair issues about the reolink integration?
What URL do you have configured under settings->network->local HA URL?
Are you using a SSL certificate and if so how is it configured in HomeAssistant?
Are you using VLANs in your network?

Partly this will be fixed by #91070, but I am doubting if ONVIF pushes are comming through correctly in your system.

@fr33pr0d
Copy link
Author

Local URL: http://192.x.x.x:8123
I'm using cloudflare for external access
No VLANs
Now it seems to be working about 50% of the time. Only doorbell press is the issue. Video stream, person, vehicle and general motion detection work just fine.

@starkillerOG
Copy link
Contributor

@fr33pr0d In that case it is most likely going to be fixed with PR #91070, please wait on that beeing released. I will notify you when it is ready and ask to confirm your issue is fixed. Probably in a few days.

@starkillerOG
Copy link
Contributor

If you appreciate the reolink integration and want to support its development, please consider sponsering the upstream library.

@fr33pr0d
Copy link
Author

@starkillerOG thanks, I'll wait for the update 👍🏻

@palmerch1
Copy link

Hmm, still seems to be very slow here when it works(takes about 10seconds before HA detects the doorbell push
(Running on HA 2023.4.4 with Doorbell v3.0.0.1859_23030902 here)
And HA is only detecting the binary_sensor.visitor 1 time out of 3.
The Reolink phone app on the other hand is reacting immediately every time.

Ps: just added myself as a sponsor, because you're doing a great job keeping this code up to date !

@starkillerOG
Copy link
Contributor

@palmerch1 thank you very much for sponsering!
The ONVIF fix has been merged and will be available in HomeAssistant 2023.4.5, that is expected to be released in the comming days.

Please hold on for a few more days untill HA 2023.4.5 is available, and please let me know if it is working once you upgraded.
If you still experiance problems, let me know and I will guide you through the process of making debug logs that I can analyse to figure out the problem.

@starkillerOG
Copy link
Contributor

HomeAssistant 2023.4.5 is now available and includes the ONVIF push fix.
Please let me know if it is now working for you on 2023.4.5.
I will close this issue, if it still does not work for you, please create a new issue.

@starkillerOG
Copy link
Contributor

@home-assistant close

@github-actions github-actions bot locked and limited conversation to collaborators May 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants