Skip to content

Fix Z2M not replying to OTA requests#32092

Draft
andrei-lazarov wants to merge 1 commit into
Koenkk:devfrom
andrei-lazarov:patch-1
Draft

Fix Z2M not replying to OTA requests#32092
andrei-lazarov wants to merge 1 commit into
Koenkk:devfrom
andrei-lazarov:patch-1

Conversation

@andrei-lazarov
Copy link
Copy Markdown
Contributor

I saw a Philips Hue bulb spamming OTA requests.

I expected Z2M to reply "No image available", so the bulb would stop asking.
Is this omitted on purpose @Nerivec?

Ctrl+F requested OTA: log.log

Screenshot From 2026-05-24 03-52-40

@andrei-lazarov andrei-lazarov changed the base branch from master to dev May 24, 2026 01:11
@Nerivec
Copy link
Copy Markdown
Collaborator

Nerivec commented May 24, 2026

I checked, it's been like this for over 6 years (I stopped going back after that much 😬).
I think this was intended to de-duplicate. If you allow replies with NO, it might interfere with the first response that could be YES. You'd basically be saying, YES, NO, NO, NO, NO... which no doubt is going to create some troubles.

@andrei-lazarov
Copy link
Copy Markdown
Contributor Author

What do you mean the first response could be yes? Isn't that only when attempting to start the update?

@Nerivec
Copy link
Copy Markdown
Collaborator

Nerivec commented May 24, 2026

There's logic on ZH side when checking, depends on several factors. But actually, I think everything replies NO in check path, so, I guess could be fine.

Thing is, we should already be replying NO to the first message, so, if that doesn't shut up the device, I don't think replying to the dupes would stop it either. We'd only end up spamming to reply to spam. 😅

@andrei-lazarov andrei-lazarov marked this pull request as draft May 25, 2026 09:39
@andrei-lazarov
Copy link
Copy Markdown
Contributor Author

I'm sniffing another network and Z2M does actually reply to all devices requesting ota 🤔 Will have to check again

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