Skip to content

Narrow the broad exception handling - #324

Merged
Sholofly merged 1 commit into
devfrom
fix/narrow-broad-excepts
Oct 30, 2026
Merged

Sholofly merged 1 commit into
devfrom
fix/narrow-broad-excepts

Conversation

@Sholofly

Copy link
Copy Markdown
Owner
  • The best-effort fetches of replay channels, the EPG and event details caught every exception, which also hid programming errors. Catch the library errors and unexpected payload shapes instead.
  • Adding an SSDP-discovered device only aborts for library errors.
  • Stop validate_config from turning every unexpected error into "cannot connect". It now falls through to the caller, which shows the unknown error and logs the traceback.
  • Add tests for the media player fetches and for the config flow errors.

- The best-effort fetches of replay channels, the EPG and event details
  caught every exception, which also hid programming errors. Catch the
  library errors and unexpected payload shapes instead.
- Adding an SSDP-discovered device only aborts for library errors.
- Stop validate_config from turning every unexpected error into
  "cannot connect". It now falls through to the caller, which shows the
  unknown error and logs the traceback.
- Add tests for the media player fetches and for the config flow errors.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@Sholofly
Sholofly enabled auto-merge (squash) October 30, 2026 22:35
@Sholofly
Sholofly merged commit 4d3e467 into dev Oct 30, 2026
2 checks passed
@Sholofly
Sholofly deleted the fix/narrow-broad-excepts branch October 30, 2026 22:36
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.

1 participant