try to reconect on some exceptions

[cogisum]

Give it a chance to reconnect on some non-fatal exceptions.

[Copilot]

Pull Request Overview

This PR modifies the WebSocket reconnection logic to handle additional exceptions as recoverable instead of fatal, allowing the connection to automatically retry rather than completely failing.

• Adds ConnectionClosedOK exception handling for graceful connection closures
• Moves BinanceWebsocketUnableToConnect from fatal to recoverable exception category

[Copilot]

The PR title contains a spelling error: 'reconect' should be 'reconnect'.

[pcriadoperez]

Hi @cogisum.

Thanks for the PR!

I believe it's correct to break the loop when throwing BinanceWebsocketUnableToConnect as this means the loop has tried to reconnect several times unsuccesfully.

As for the ConnectionClosedOK exception, I believe it's also correct to break the loop as the connection was closed cleanly.

Could you explain a bit more the issue you are trying to solve?

[cogisum]

Hi @pcriadoperez , the PR is for ConnectionClosedOK, which I have met many times. The closing action was not initiated by me, so I think recovery needed.