Skip to content

Clarify PoolLab API timeout and empty error messages - #70

Open
hmmbob wants to merge 2 commits into
dala318:mainfrom
hmmbob:fix/poollab-error-messages
Open

hmmbob wants to merge 2 commits into
dala318:mainfrom
hmmbob:fix/poollab-error-messages

Conversation

@hmmbob

@hmmbob hmmbob commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Exceptions such as TimeoutError() have an empty string representation, so PoolLab currently logs Unknown error communicating with API: without identifying the failure.

Report request timeouts explicitly and explain automatic retry. The message is independent of the configured deadline, since a timeout may originate in either the HTTP/GraphQL client or the coordinator. For other failures, retain the exception type and provide fallback text when its message is empty. Authentication handling and polling behavior are unchanged.

Validation:

  • Four focused regression tests pass with Home Assistant 2026.10.1 / Python 3.14.
  • The timeout and empty-message cases reproduce the uninformative message against the original code.
  • The full suite cannot collect tests/test_coordinator.py in this environment because its existing ConfigEntry construction lacks subentries_data. The same collection error occurs on unchanged upstream code; this PR leaves that separate test compatibility issue untouched.
  • git diff --check passes.

The error-message change can be merged independently of #73: it does not change the timeout setting or require a timeout constant. The real deadline regression test expires the timeout context without depending on its configured duration.

schford added a commit to schford/python_poollab that referenced this pull request Oct 22, 2026
schford added a commit to schford/python_poollab that referenced this pull request Oct 22, 2026
- hacs.json: drop zip_release/filename (no release workflow runs on the fork)
- manifest.json: version 1.3.1-sf1
- README: fork note naming the carried upstream PRs dala318#70-dala318#73

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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