Disable webhooks on cameras that have old firmwares - after 2023.5.X some old ONVIF cams stopped working

lucky · May 15, 2023, 6:45pm

disable webhooks on cameras that have old firmwares

linked to:

github.com/home-assistant/core

IP CAM ESCAM G02 C9F0SeZ0N0P4L0 firmware crashes when creating webhook subscription after updating HA to 2023.5.2

opened 10:29PM - 13 May 23 UTC

luckyzor

problem in device integration: onvif

### The problem No video Image of my CAM. it works in the Onvif Manager. ![…image](https://github.com/home-assistant/core/assets/31155573/2894133d-8bec-4671-904a-1b3d1f947afd) ![image](https://github.com/home-assistant/core/assets/31155573/ca04a021-a145-4a9d-ad52-bfb371f3a207) ![image](https://github.com/home-assistant/core/assets/31155573/d46a9842-bcd8-4370-8189-6e3b48ed0738) ### What version of Home Assistant Core has the issue? core-2023.5.2 ### What was the last working version of Home Assistant Core? Home Assistant 2023.4.X ### 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-073c6921cdff27033b610698b1464da3.json (1).txt](https://github.com/home-assistant/core/files/11470767/config_entry-onvif-073c6921cdff27033b610698b1464da3.json.1.txt) ### Example YAML snippet _No response_ ### Anything in the logs that might be useful for us? _No response_ ### Additional information _No response_

bdraco · May 16, 2023, 8:47pm

github.com/home-assistant/core

Add option to disable webhooks in ONVIF

home-assistant:dev ← home-assistant:onvif_webhooks_option

opened 08:44PM - 16 May 23 UTC

bdraco

+50 -3

TODO: - [ ] config flow tests - [x] more manual testing ## Proposed ch…ange  feature request: https://community.home-assistant.io/t/disable-webhooks-on-cameras-that-have-old-firmwares-after-2023-5-x-some-old-onvif-cams-stopped-working/571854 related issues: #92892 #93043 #92943 ## Type of change  - [ ] Dependency upgrade - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New integration (thank you!) - [x] New feature (which adds functionality to an existing integration) - [ ] Deprecation (breaking change to happen in the future) - [ ] Breaking change (fix/feature causing existing functionality to break) - [ ] Code quality improvements to existing code or addition of tests ## Additional information  - This PR fixes or closes issue: fixes # - This PR is related to issue: - Link to documentation pull request: ## Checklist  - [x] The code change is tested and works locally. - [ ] Local tests pass. **Your PR cannot be merged unless tests pass** - [ ] There is no commented out code in this PR. - [ ] I have followed the [development checklist][dev-checklist] - [ ] I have followed the [perfect PR recommendations][perfect-pr] - [ ] The code has been formatted using Black (`black --fast homeassistant tests`) - [ ] Tests have been added to verify that the new code works. If user exposed functionality or configuration variables are added/changed: - [ ] Documentation added/updated for [www.home-assistant.io][docs-repository] If the code communicates with devices, web services, or third-party tools: - [ ] The [manifest file][manifest-docs] has all fields filled out correctly. Updated and included derived files by running: `python3 -m script.hassfest`. - [ ] New or updated dependencies have been added to `requirements_all.txt`. Updated by running `python3 -m script.gen_requirements_all`. - [ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description. - [ ] Untested files have been added to `.coveragerc`.  To help with the load of incoming pull requests: - [ ] I have reviewed two other [open pull requests][prs] in this repository. [prs]: https://github.com/home-assistant/core/pulls?q=is%3Aopen+is%3Apr+-author%3A%40me+-draft%3Atrue+-label%3Awaiting-for-upstream+sort%3Acreated-desc+review%3Anone+-status%3Afailure  [dev-checklist]: https://developers.home-assistant.io/docs/development_checklist/ [manifest-docs]: https://developers.home-assistant.io/docs/creating_integration_manifest/ [quality-scale]: https://developers.home-assistant.io/docs/integration_quality_scale_index/ [docs-repository]: https://github.com/home-assistant/home-assistant.io [perfect-pr]: https://developers.home-assistant.io/docs/review-process/#creating-the-perfect-pr