Sanae6 / SmoOnlineServer

Official server for Super Mario Odyssey: Online
https://discord.gg/jYCueK2BqD
103 stars 24 forks source link

fix: don't output empty player IDs or RemoteEndPoints in the log #36

Closed Istador closed 1 year ago

Istador commented 1 year ago

Make and use a copy of the RemoteEndPoint at the start of the HandleSocket method. Because in some cases when the socket is disposed, the RemoteEndPoint inside of it is cleared and isn't available for the following disconnect log entries. Also: port scanners on the internet don't introduce themselves with a name and ID.

(cherry picked from commit 2f4cd0509a683f44eff7b96b1b88efaf65485fb6)