Skip to content

Fix disarming linkage 400 error on cameras without channel-indexed config - #562

Merged
rroller merged 1 commit into
rroller:mainfrom
irfadrazick:fix/disarming-linkage-no-channel-index
Apr 1, 2026
Merged

Fix disarming linkage 400 error on cameras without channel-indexed config#562
rroller merged 1 commit into
rroller:mainfrom
irfadrazick:fix/disarming-linkage-no-channel-index

Conversation

@irfadrazick

Copy link
Copy Markdown
Contributor

Summary

  • Some cameras (e.g. DH-P3D-3F-PV-P) return 400 Bad Request when setting DisableLinkage and DisableEventNotify with a channel index, even though the read (getConfig) succeeds
  • Fall back to the non-indexed URL if the channel-indexed setConfig call fails

Test plan

  • Tested on DH-P3D-3F-PV-P (firmware 3.000.0000000.6.R, build:2025-09-25)
  • Toggling Disarming and Event Notifications switches in Home Assistant now works without error

🤖 Generated with Claude Code

…nfig

Some cameras (e.g. DH-P3D-3F-PV-P) return 400 Bad Request when setting
DisableLinkage and DisableEventNotify with a channel index. Fall back to
the non-indexed URL if the indexed call fails.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@irfadrazick

Copy link
Copy Markdown
Contributor Author

Hey @rroller, I tested this change with my camera by modifying the code directly in my HA and actively using it. It would be great if you could check and merge it so I can install the latest updates of the integration when you release.

@rroller

rroller commented Mar 30, 2026

Copy link
Copy Markdown
Owner

Thanks! Will get this merged soon. Sorry been busy

@rroller
rroller merged commit 94bc275 into rroller:main Apr 1, 2026
3 checks passed
@rroller

rroller commented Apr 1, 2026

Copy link
Copy Markdown
Owner

Released!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants