Skip to content

Add connectionAbortedError handling#796

Merged
1Blademaster merged 2 commits intomainfrom
795-connection-aborted-error-not-handled
Oct 26, 2025
Merged

Add connectionAbortedError handling#796
1Blademaster merged 2 commits intomainfrom
795-connection-aborted-error-not-handled

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

@1Blademaster 1Blademaster requested review from a team and Copilot October 26, 2025 14:27
@1Blademaster 1Blademaster linked an issue Oct 26, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

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 PR adds specific handling for ConnectionAbortedError exceptions when checking for messages from the autopilot. Previously, these errors would have been caught by the generic Exception handler.

Key changes:

  • Added a dedicated exception handler for ConnectionAbortedError that logs the error, invokes the disconnect callback, and closes the connection

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread radio/app/drone.py Outdated
@1Blademaster 1Blademaster merged commit 3492761 into main Oct 26, 2025
7 checks passed
@1Blademaster 1Blademaster deleted the 795-connection-aborted-error-not-handled branch October 26, 2025 17:24
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.

Connection aborted error not handled

3 participants