Unhandled exception in opened handler (SocketIO v2.3.1)

Hi @doghappy Customer sent screenshot with unhandled exception. and brief exception text part:

************** Exception Text ************** SocketIOClient.Exceptions.InvalidSocketStateException: Connection is not open. at SocketIOClient.WebSocketClient.ClientWebSocket.<SendMessageAsync>d__18.MoveNext() — End of stack trace from previous location where exception was thrown — at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at SocketIOClient.WebSocketClient.ClientWebSocket.<SendMessageAsync>d__17.MoveNext() — End of stack trace from previous location where exception was thrown — at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at SocketIOClient.SocketIO.<OpenedHandler>d__105.MoveNext() — End of stack trace from previous location where exception was thrown — at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()

Seems exception occurs in OpenedHandler method at attempt to execute await Socket.SendMessageAsync(msg);

        private async void OpenedHandler(string sid, int pingInterval, int pingTimeout)
        {
            Id = sid;
            _pingInterval = pingInterval;
            string msg = Options.EioHandler.CreateConnectionMessage(Namespace, Options.Query);
            await Socket.SendMessageAsync(msg);
        }

Exception was not caught in OpenedHandler() or in OpenedProcessor.Process, which called this method as delegate. Can you please give recommendation how this situation is possible and how to fix code in 2.3.1 (this lib version is used in production and should be fixed).

[doghappy]

Seems exception occurs in OpenedHandler method at attempt to execute await Socket.SendMessageAsync(msg);

If the exception is indeed thrown here, it means that the WebSocket was disconnected shortly after the connection was established, causing some information of socket.io to not be sent.

If this problem occurs by accident, can it be fixed like this?

        private async void OpenedHandler(string sid, int pingInterval, int pingTimeout)
        {
            Id = sid;
            _pingInterval = pingInterval;
            string msg = Options.EioHandler.CreateConnectionMessage(Namespace, Options.Query);
            try
            {
                throw new Exception("TEST Exception");
                // await Socket.SendMessageAsync(msg).ConfigureAwait(false);
            }
            catch (Exception e)
            {
                ErrorHandler(e.Message);
                await ConnectAsync().ConfigureAwait(false);
            }
        }

btw, I will consider adding a better solution to the latest version.

[DmitryBk]

It works for me with throw new Exception(“TEST Exception”); It seems the OpenedHandler is called after the connection is established and after ConnectedHandler which set Connected = true

Sorry I was wrong, Connected is really false in that moment. My new suggestion just increments Attempts that OnReconnected event will be faired.

catch (Exception e)
{
    ErrorHandler(e.Message);

    Attempts++;
    await ConnectAsync().ConfigureAwait(false);
}