WatsonTcpServer.ListClients() is empty if called "too early"

Hey!

server.Start();
client.Connect();

Console.WriteLine(server.ListClients().Count());
await Task.Delay(1); // Thread.Sleep(1) also works
Console.WriteLine(server.ListClients().Count());

This will output:

0
1

This is very problematic as one might not expect server.ListClients() to return an empty enumerable, especially if they’re not aware of this issue.

I’m not sure why a 1ms delay seems to fix it. Are some async events (registering the client on server side) not awaited/fired on time or something?

Why does this happen and could it be fixed?

[Laiteux]

Yeah, very understandable.

You can label this issue if you wish and keep it for later or whatever, I guess it’ll be fine for now. Especially if I’m the first one reporting about it.

In a real server<->client scenario, this probably won’t be important anyway because of the fact there is already ping between everything.

This was found because I ran some tests locally, but I can hardly think of a situation where you’d need to call ListClients() less than a few milliseconds after a client connects.

[jchristn]

Yeah, not sure how I’m going to be able to fix that. There’s very little code between the server accepting the connection and the client metadata being added to the client list. This is a biproduct of the fact that the client is returning faster from BeginConnect() (and everything else it does within Connect() than the server does from the first few steps of AcceptConnections().