Skip to content

Conversation

@dimitrisn442
Copy link
Contributor

Changed (1)

  • Enhanced WebSocket reconnection logic with intelligent error classification.

learnerLj and others added 5 commits July 23, 2025 04:30
…assification

- Add ConnectionFailureReason enum to classify WebSocket failures
- Implement smart reconnection decisions based on error types
- Handle network errors and stream ending scenarios that were previously ignored
- Distinguish between recoverable errors (network issues, temporary server errors)
  and permanent failures (authentication, protocol violations)
- Improve writer task error handling with reconnection capability
- Add comprehensive error classification for WebSocket close codes per RFC 6455
- Enhance logging with detailed error classification for better monitoring
- Ensure financial-grade reliability by avoiding unnecessary reconnection attempts

All 162 existing tests pass, maintaining backward compatibility.
…d add comprehensive tests

- Move ConnectionFailureReason from websocket.rs to errors.rs
- Rename to WebsocketConnectionFailureReason for better clarity
- Add comprehensive test coverage for error classification logic
- Consolidate duplicate match arms to fix clippy warnings
- Update all imports and usages throughout codebase
- Tests cover tungstenite error mapping, close code classification, and reconnection logic

Addresses PR feedback requirements for better code organization and test coverage.
feat(websocket): enhance reconnection logic with intelligent error classification
@dimitrisn442 dimitrisn442 merged commit f54679c into main Jul 29, 2025
3 of 4 checks passed
@dimitrisn442 dimitrisn442 deleted the rc-v11.0.0 branch July 29, 2025 06: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.

4 participants