Skip to content

Handle errors gracefully for non-critical data requests - #7693

Merged
ryaplots merged 3 commits into
v3.34from
fix/error-handling
Aug 27, 2025
Merged

Handle errors gracefully for non-critical data requests#7693
ryaplots merged 3 commits into
v3.34from
fix/error-handling

Conversation

@ryaplots

Copy link
Copy Markdown
Contributor

Summary

This PR is a followup of the incident we had during the weekend.
The console does a lot of requests on init, which are not crucial for it to work, so I have logged the errors and just marked the data as unavailable for the time-being, that way the console stays functional. For more important requests, like managed gateway or gateway stats this only happens if the request was aborted with a timeout.

Changes

  • Handle gracefully errors on init
  • Show info as unavailable for gateways stats and managed gateway requests

Testing

Steps

mock an init request to time out and see that you are still able to log in to the console.
mock the gateway stats request to time out and observe the gateway status panel shows a message.

Results
Screenshot 2025-08-12 at 15 38 54

Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Compatibility: The changes are backwards compatible with existing API, storage, configuration and CLI, according to the compatibility commitments in README.md for the chosen target branch.
  • Documentation: Relevant documentation is added or updated.
  • Testing: The steps/process to test this feature are clearly explained including testing for regressions.
  • Infrastructure: If infrastructural changes (e.g., new RPC, configuration) are needed, a separate issue is created in the infrastructural repositories.
  • Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.
  • Commits: Commit messages follow guidelines in CONTRIBUTING.md, there are no fixup commits left.

@ryaplots ryaplots added this to the v3.35.0 milestone Aug 12, 2025
@ryaplots ryaplots self-assigned this Aug 12, 2025
@ryaplots
ryaplots requested a review from a team as a code owner August 12, 2025 13:39
@ryaplots ryaplots added the ui/web This is related to a web interface label Aug 12, 2025
@ryaplots
ryaplots requested a review from mjamescompton August 12, 2025 13:39
@ryaplots
ryaplots merged commit ef3a5cc into v3.34 Aug 27, 2025
12 of 13 checks passed
@ryaplots
ryaplots deleted the fix/error-handling branch August 27, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui/web This is related to a web interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants