Skip to content

fix: ignore non-GlobalCache AMXB discovery beacons (#12) - #13

Merged
sickkick merged 1 commit into
mainfrom
fix/12-filter-non-gc-amxb
Sep 6, 2026
Merged

fix: ignore non-GlobalCache AMXB discovery beacons (#12)#13
sickkick merged 1 commit into
mainfrom
fix/12-filter-non-gc-amxb

Conversation

@sickkick

@sickkick sickkick commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Filter AMXB UDP beacons in parse_beacon so only Make=GlobalCache (case-insensitive) proceeds to discovery/config-flow validation
  • Stops Denon/Onkyo and other shared-AMXB devices from triggering recurring Could not connect …:4998 warnings (iTach Validation Failures #12)
  • Adds unit tests for foreign Make, missing Make, and case-insensitive acceptance

Test plan

  • python3 -m pytest tests/test_discovery.py -v (11 passed)
  • On HA with Global Cache + Denon/Onkyo AMXB devices: confirm foreign beacons no longer appear as discovery attempts / log warnings
  • Confirm a real iTach/Flex still discovers and validates normally

Fixes #12

Made with Cursor

Reject Denon/Onkyo and other shared AMXB vendors before TCP validation
so discovery no longer spams cannot_connect warnings.
@sickkick
sickkick force-pushed the fix/12-filter-non-gc-amxb branch from f68c311 to a7682a8 Compare September 6, 2026 02:01
@sickkick sickkick mentioned this pull request Sep 6, 2026
@sickkick
sickkick merged commit 7af4182 into main Sep 6, 2026
2 checks passed
@sickkick
sickkick deleted the fix/12-filter-non-gc-amxb branch September 6, 2026 02:04
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.

iTach Validation Failures

1 participant