-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
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
Failed to fetch ONVIF PullPoint subscription messages with IESS NVR (rebranded TVT) #90916
Comments
Hey there @hunterjm, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) onvif documentation |
This comment was marked as off-topic.
This comment was marked as off-topic.
It may be that your cameras / device doesn't support PullPoint. The spec only requires Notifications (webhooks) or PullPoint #91485 will add support for webhooks |
Before 2023.4 no motion sensors were added, with 2023.4 I have the motion sensors, now, with 2023.5 all sensors are unavailable. |
Try power cycling the camera and than restart HA. There is currently a problem where subscriptions are held for too long so the camera can run out of slots when upgrading/restarting HASS which isn't fixed until #91485 gets merged and released |
Tried also to remove and add the integration again but now the motion
sensors are missing.
Il giorno mar 18 apr 2023 alle ore 09:17 J. Nick Koston
***@***.***> ha scritto:
…
It may be that your cameras / device doesn't support PullPoint. The spec only requires Notifications (webhooks) or PullPoint
#91485 will add support for webhooks
Before 2023.4 no motion sensors were added, with 2023.4 I have the motion sensors, now, with 2023.5 all sensors are unavailable.
Try power cycling the camera and than restart HA
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Can you try 2023.5.2? |
Motion sensors came back also with 5.1 but even in 5.2, once detected, the
sensors remain in ‘detected’ status.
…On Fri, 5 May 2023 at 23:09, J. Nick Koston ***@***.***> wrote:
Can you try 2023.5.2?
—
Reply to this email directly, view it on GitHub
<#90916 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A5CAQBF7HODMV3DXXXHP5NTXEVUAZANCNFSM6AAAAAAWVE7KPA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Can you enable debug logs for # Example configuration.yaml entry
logger:
default: info
logs:
homeassistant.components.onvif: debug
onvif: debug
zeep: debug Restart HA Reload the config entry and than post the log |
It looks like the log was cut off. Can you send the timestamps after 2023-05-06 17:43:34.788? |
Re-attached new log compressed.
[home-assistant_2023-05-06T16-22-21.590Z.log.zip](https://github.com/home-assistant/core/files/11414850/home-assistant_2023-05-06T16-22-21.590Z.log.zip)
I made a check with the NVR logs and the webhooks received: I found that
1/2 second after the motion is detected at the NVR, HA receives the event.
Then a few seconds later there is a new event (but no log from the NVR).
Il giorno sab 6 mag 2023 alle ore 19:08 J. Nick Koston <
***@***.***> ha scritto:
… It looks like the log was cut off. Can you send the timestamps after
2023-05-06 17:43:34.788?
—
Reply to this email directly, view it on GitHub
<#90916 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A5CAQBFA3PT5N6XZCSW47TDXE2AQXANCNFSM6AAAAAAWVE7KPA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
So it is getting the webhooks Thats good. |
Can you try the updated version with
Verify that the version displayed above is "onvif-zeep-async==3.1.1" and NOT 2.1.1 Restart HA Test Delete |
I'm now with the custom components I can confirm "onvif-zeep-async==3.1.1". Atteched the new log. The behaviour is the same: detected ok but stay forever. I noticed in the logs that it seems with this version not using webhooks anymore. |
interesting
|
That looks ok as its only doing it once per camera |
Its accepting the subscription |
pull point looks good as well |
I don't see it even trying to pull the events though |
I made another change to the PR (but not the library). It should force it to pull
Verify that the version displayed above is "onvif-zeep-async==3.1.1" and NOT 2.1.1 Restart HA Test Delete |
I just made another fix so if you have run the above already, please do it again and restart again |
I did it, same behaviour, sensors remains in 'detected' mode. Just a question: what should be the right behaviour? I mean, once the NVR sends the motion event, should the NVR/Camera send a 'clear' event? Why not immediately clear after the event? |
It usually clears about 30 seconds after depending on how they implemented it. Some NVRs don't actually implement the clear event which might be the case here. |
The subscription setup looks good..
I made another change to the PR and the library
Verify that the version displayed above is "onvif-zeep-async==3.1.2" and NOT 2.1.1 Restart HA Test Delete |
Attached DEBUG with latest. Same behaviour. Maybe no clear event at all from NVR. If it is the case, how to clear the sensor? |
I don't actually see Home Assistant trying to pull the event messages in the log but it may just be that the log was cut off too soon.
The NVR has to send a |
Do you have a link to the NVR's docs/where you got it from/where to buy it? |
Unfortunately it is only in italian and not updated. In any case there is no explicit option to enable events but to enable ONVIF and this is what I did adding a specific user for HA. Infact after this I was able to add the NVR with the ONVIF integration. As it sends the motion event I don't think there should be anything else to enable (otherwise neither the motion event had to work). |
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. |
The problem
I've a ONVIF compatible NVR (IESS that is a rebranding of TVT) that prior to 2023.4.0 gave no error but only exposed the video streams (no motion detection sensors). With 2023.4.0 the integration added several sensors including motion sensors but now I've the log filled. The full DEBUG log is added to diagnostic data.
What version of Home Assistant Core has the issue?
core-2023.4.0
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
ONVIF
Link to integration documentation on our website
https://www.home-assistant.io/integrations/onvif
Diagnostics information
config_entry-onvif-61dde3b5abff48c5bd7f14162ec9d2f8.json.txt
home-assistant_onvif_2023-04-06T09-06-11.392Z.log.zip
Example YAML snippet
No response
Anything in the logs that might be useful for us?
Additional information
In 2023.4.0 is the first time that finally I have motion sensor enabled
The text was updated successfully, but these errors were encountered: