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

[ipcamera] ONVIF events are processed faster, and adds a choice of what event method will be used. #17732

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

TheNetStriker
Copy link
Contributor

@TheNetStriker TheNetStriker commented Nov 11, 2024

I made various changes and fixes to the ONVIF connection in the IpCamera binding. I already discussed those changes in this community thread.

The addon now checks if there is support for PullPoint and WSBaseNotification subscriptions. PullPoint subscriptions are prefered because we have more control over it. I also added an Thing option to force a specific ONVIF event service type.

This also fixes an error with older Reolink cameras where the scheduleVersion is null in the Reolink API.

@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/ipcamera-fixes-and-optimizations-to-onvif-connection/160097/12

@Skinah Skinah changed the title [ipcamera] ONVIF fixes for IpCamera [ipcamera] ONVIF events can be processed faster, and adds a choice of what event method will be used. Nov 11, 2024
@Skinah Skinah added the enhancement An enhancement or new feature for an existing add-on label Nov 11, 2024
@Skinah Skinah changed the title [ipcamera] ONVIF events can be processed faster, and adds a choice of what event method will be used. [ipcamera] ONVIF events are processed faster, and adds a choice of what event method will be used. Nov 11, 2024
@Skinah
Copy link
Contributor

Skinah commented Nov 11, 2024

Thank you for the changes. Just need to get the DCO happy when you have time. LGTM.

@TheNetStriker
Copy link
Contributor Author

Thank you for the changes. Just need to get the DCO happy when you have time. LGTM.

I already signed off the commit, but I guess the problem is that I had not set my real name in my GitHub profile. (Regarding the contribution guidelines I have to sing off with my real name)

I just added my real name to my GitHub profile. Can you manually trigger the DCO to run again? I hope it works after that.

@lsiepel
Copy link
Contributor

lsiepel commented Nov 11, 2024

Thank you for the changes. Just need to get the DCO happy when you have time. LGTM.

I already signed off the commit, but I guess the problem is that I had not set my real name in my GitHub profile. (Regarding the contribution guidelines I have to sing off with my real name)

I just added my real name to my GitHub profile. Can you manually trigger the DCO to run again? I hope it works after that.

Next to the DCO check is a 'details' link that shows a page on how to rewrite the signoff message. You could also create an arbitrary commit with the right signoff (like merge master into your feature branch)

I made various changes and fixes to the ONVIF connection in the IpCamera binding. I already discussed those changes in [this](https://community.openhab.org/t/ipcamera-fixes-and-optimizations-to-onvif-connection/160097) community thread.

The addon now checks if there is support for PullPoint and WSBaseNotification subscriptions. PullPoint subscriptions are prefered because we have more control over it. I also added an Thing option to force a specific ONVIF event service type.

This also fixes an error with older Reolink cameras where the scheduleVersion is null in the Reolink API.

Signed-off-by: David Masshardt <[email protected]>
@TheNetStriker
Copy link
Contributor Author

Thank you for the changes. Just need to get the DCO happy when you have time. LGTM.

I already signed off the commit, but I guess the problem is that I had not set my real name in my GitHub profile. (Regarding the contribution guidelines I have to sing off with my real name)
I just added my real name to my GitHub profile. Can you manually trigger the DCO to run again? I hope it works after that.

Next to the DCO check is a 'details' link that shows a page on how to rewrite the signoff message. You could also create an arbitrary commit with the right signoff (like merge master into your feature branch)

Thanks, I was able to trigger the DCO again.

@lsiepel lsiepel merged commit 66f8c82 into openhab:main Nov 11, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.3 milestone Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants