r002 / rittr

A Robertlandian take on Twitter...
0 stars 0 forks source link

**Fix bug:** If a "normal" client connects, the "Prune-Zombie-Clients" toggle setting is being ignored! #32

Closed r002 closed 4 years ago

r002 commented 4 years ago

MONDAY

Fix bug: If a "normal" client connects, the "Prune-Zombie-Clients" toggle setting is being ignored!

r002 commented 4 years ago

Fixed.

If the "Auto-Prune-Zombie Clients" toggle is disabled, the server will broadcast a new clientMap every time a new client connects. But it will not heartbeat all connected clients. This means zombie clients will continue to be in the clientMap.

image

image


Btw, one potential side-effect though:

If the loop toggle is disabled, any new clients that connect will not receive any heartbeat requests after connecting. This means that in their foot, it simply shows Initializing Connection: ⌛" forever which is not ideal. 🤔🤔🤔

image