adjust server global active connection counter to consider multiplexing

[0xb4lamx]

This PR updates the server getConnectionCount() to accurately handle multiplexed connections by considering unique socketIds. The previous implementation could overcount connections when multiple documents shared the same WebSocket connection.

[0xb4lamx]

up! if someone could have a look here 👀

[janthurau]

@0xb4lamx thanks! I'm currently on holiday, sorry for the long wait. I've approved the pipeline but two tests seem to need adjustments, as we're asserting the connection count there. I agree the connection count should count per real connection (i.e. not count multiplexing connections n times), so happy to merge once fixed :-)

[0xb4lamx]

@0xb4lamx thanks! I'm currently on holiday, sorry for the long wait. I've approved the pipeline but two tests seem to need adjustments, as we're asserting the connection count there. I agree the connection count should count per real connection (i.e. not count multiplexing connections n times), so happy to merge once fixed :-)

Thanks for your time! should be fixed in f691c1b 👀

[janthurau]

Thank you! 😍