Connection UID

[Cyclone]

Hi, great code! I wonder if is it possible to make a uid for each connected client for database purpose like websocket-sharp for C#.
is is possible? If not can you consider it as a TODO?

Also having a OnError event would be nice.

[Davidson Francis]

Hi @CycloneRing,

It seems interesting, but what is the use case? Since users are 'ephemeral', it seems to me that using the 'fd' of the connection as an ID would be enough, wouldn't it?

About OnError, I haven't added it yet as it seems unnecessary to me. However, to make it useful, adding an error reason, such as 'handshake failure' or 'malformed frame' would be necessary.

[Cyclone]

@Theldus Thank you for your reply, Currently clientID is the pointer of array, It means if a connection disconnect and someone else connect it get same previous ID which last client disconnected from. Having a UID give better advantage on identifying clients.

[Davidson Francis]

Hi @CycloneRing,
Thanks for coming back on this issue.

Yes, you are right, indeed, this is a problem that I have been thinking about how to solve for a long time.

The worst problem with the current approach is that ws_sendframe*() can't guarantee that the message is delivered to the right client, and this is quite problematic, especially if you want to save the clients list (as in #70) or handle clients later by any reason.

Thinking about implementation maybe a simple approach would be:

• ws_cli_conn_t is no longer an opaque pointer, but an alias to 'uint64_t' that stores the "CID" (Client ID).
• The CID can be as simple as a client counter, always incremented (to ensure uniqueness).
• All of the routines that previously received the pointer directly from the connection (ws_cli_conn_t *) now receive the "CID" and iterate over the clients list to find the client corresponding to that ID. If the client cannot be found, it no longer exists.

What do you think about? this shouldn't be too complicated to implement and should solve all the issues mentioned before.

[Cyclone]

Thank you for your time, It makes sense but why not just using a struct based array instead of actual pointer like
struct ClientObject
{
ws_cli_conn_t clientPtr;
uint64_t clientID;
int clientIndex;
}

should work?

[Davidson Francis]

Hi @CycloneRing,
Actually I am referring to what should be passed as a parameter to the on_open (and related) functions and also in ws_sendframe_*(), like in:

void onmessage(<something>, const unsigned char *msg,
    uint64_t size, int type)
{
  ws_sendframe_txt(<something>, "message");
}

So I don't really know how to fit a struct like you proposed: if it's static, I need to reuse it for another client later, if it's dynamic, I need to deallocate it at some point.

In the current code, ws_cli_conn_t refers to an internal wsServer structure, which as you noticed, can change and point to another client later. So dealing with another similar data structure looks like I'm going to repeat the same problem...

[Cyclone]

Hi @Theldus , I will put some time on this later to see if I can implement it and let you know.
Thanks for your dedication.