Skip to content

fix: remove backoff in case of failures#79

Merged
lhw merged 1 commit intomainfrom
fix/remove-backoff
Feb 27, 2026
Merged

fix: remove backoff in case of failures#79
lhw merged 1 commit intomainfrom
fix/remove-backoff

Conversation

@lhw
Copy link
Copy Markdown
Owner

@lhw lhw commented Feb 22, 2026

Weathercloud seems unreliable. And without an option to re-enable the sending this is just annoying.

Also its at most one request per minute. So whatever

Copilot AI review requested due to automatic review settings February 22, 2026 12:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request removes the proxy failure tracking and automatic backoff mechanism that was previously disabling weather service proxies after repeated failures. The change is motivated by Weathercloud being unreliable and the fact that weather stations send updates infrequently (at most once per minute), making permanent disabling of forwarding problematic when there's no way to re-enable it without reconfiguration.

Changes:

  • Removed failure tracking data structures (proxy_failure_counts and disabled_proxy_sinks)
  • Simplified error handling to log failures without counting or disabling proxies
  • Improved session.closed handling to allow retries on subsequent requests rather than permanent disabling

@lhw lhw merged commit 8fedd2e into main Feb 27, 2026
7 checks passed
@lhw lhw deleted the fix/remove-backoff branch February 27, 2026 19:18
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