[BUG] `Disconnected` event gets incorrect values

Issue Type

• Bug

Describe the bug The Disconnected event returns only values from WebSocketCloseStatus (1000-1011) and marks them as ObsCloseCodes. Whereas the OBS protocol returns the values 0, 4000, 4002, 4003-4012.

To Reproduce Steps to reproduce the behavior:

1. Connect to OBS
2. Place the breakpoint on OBSWebsocket.OnWebsocketDisconnect()
3. Close OBS
4. Check the variable d.CloseStatus

Expected behavior Probably I would like to get either a combined enum from the OBS protocol and the WebSocket enum, or get some kind of remap of WebSocket values to the values from the OBS protocol enum.

Screenshots

Versions OBS Version: 28.0.1 OBS WebSocket Version: OBS WebSocket Dotnet (this library) Version: 257e7b48e1c365fcf2ea3de9f156ded2bf8f5247 OS: Windows 11

[BarRaider]

I will address this moving forward

[ProbablePrime]

I too am seeing 1000, 1001 etc coming in this handler from e.ObsCloseCode

In my case, connect to OBS, then force close OBS -> 1001.

You asked for a reproduction:

using OBSWebsocketDotNet;
using OBSWebsocketDotNet.Communication;

namespace OBSWebsocketTest
{
    public class Program
    {
        public static Task Main(string[] args) => new Program().MainAsync();
        public async Task MainAsync()
        {
            OBSWebsocket obs = new OBSWebsocket();

            obs.Connected += Obs_Connected;
            obs.Disconnected += Obs_Disconnected;
            obs.ConnectAsync("ws://127.0.0.1:4455", "");

            // Block this task until the program is closed.
            await Task.Delay(-1);
        }
        void Obs_Connected(object? sender, EventArgs e)
        {
            Console.WriteLine("Connected");
        }

        void Obs_Disconnected(object? sender, ObsDisconnectionInfo e)
        {
            Console.WriteLine(e.ObsCloseCode);
        }
  

Running this, with OBS open, waiting till you see connected and then closing OBS will result in a 1001 close code.

I think the original poster is saying that the additional codes(<4000) are not documented in the library and that the type(ObsCloseCodes) does not handle them. This leads to a confusing experience when consuming this client.

I guess that when I consume a client application, I expect an Enum value to not contain values that are not defined in the Enum.

If I were writing this library I would probably either:

1. Extends ObsCloseCodes from another Enum type that does contain codes <4000
2. Change the handling/invocation of the disconnected handler to not send <4000 codes via the ObsCloseCodes type.

In my case, I can handle this with some extra code similar to the original poster, but I do agree with them it is confusing.