NetPlay: player timeout / handling of dead connections takes too long about nullpomino HOT 9 CLOSED

I added some sort of timeout system in r563. It still needs some testing.
Also, the check is done only when someone tries to login.

this indeed needs more testing, would be good if we can get this fixed for 7.5
I set the value to 5 sec, which seems reasonable - which causes random 
disconnects, directly in the middle of actually playing 40L games on netplay.
I'll keep it at 5000 for now, that you can debug this.

ok I tested some more and saw in the server logs that these random disconnects 
seem to be realted to new players login in the server, e.g.:

00:36:44,196 [main] INFO  NetServer: Logout: **ip**
00:36:44,196 [main] INFO  NetServer: xlro !NANNANNANA has logged out
00:36:44,215 [main] INFO  NetServer: RoomDelete ID:18 Title:Single:LINE RACE
00:36:44,224 [main] INFO  NetServer: Logout: **ip**
00:36:44,224 [main] INFO  NetServer: Cedevita has logged out
00:36:44,224 [main] INFO  NetServer: Killed 2 dead connections
00:36:44,226 [main] INFO  NetServer: Paradox  !botJudpywY has logged in 
(Host:**ip** Team:)
00:36:44,981 [main] INFO  NetServer: Hyper Zircean !Ya.GWW5jkg has logged in 
(Host:**ip** Team:)

I will increase the timeout delay now till the issue doesnt show up anymore, 
since the server is not playable like this, as soon as someone joins others get 
dropped

let me know if you want to debug this on nullpo.it.cx, then I can change the 
settings for you

just to note and rule out any coincidences: I also increased 
netserver.maxLobbyChatHistory and netserver.maxRoomChatHistory when I lowered 
netserver.timeoutTime

5 seconds? That's too short.
Currently NullpoMino sends a ping message every 10 seconds. So 
random-disconnection will happen if you set the timeout below 10 seconds.
Maybe we should set ping interval to 5 or less seconds?

okay, I was under the impression from the description that the timeoutTime 
value defined the ping/pong interval and that this is not hardcoded. Maybe add 
it as another config value? 5 sec makes sense to me, I have it set to 10 atm, 
and it seems to work ok. But giving this value to the server admin to control, 
is pbbly best.

That's a good idea.
I added server-controlled Ping interval option in r618. Please try it.

ok great, updated the server to r618, running with

netserver.timeoutTime=15000
netserver.clientPingInterval=5000

now for testing - looks ok so far.