edmand46/Ragon
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixed: double authorization request

Browse Source
This commit is contained in:
Edmand46
2022-10-15 18:48:59 +04:00
parent 9a22566f79
commit c2c75cb513
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Ragon/Sources/Lobby/Lobby.cs
+11 -4
View File
@@ -28,15 +28,22 @@ public class Lobby
_serializer.Clear(); _serializer.Clear();
_serializer.FromSpan(ref payload); _serializer.FromSpan(ref payload);
var player = _authorizationManager.GetPlayer(peerId);
if (op == RagonOperation.AUTHORIZE) if (op == RagonOperation.AUTHORIZE)
{ {
var key = _serializer.ReadString(); if (player != null)
var playerName = _serializer.ReadString(); {
_authorizationManager.OnAuthorization(peerId, key, playerName); _logger.Warn("Player already authorized");
return;
}
var key = _serializer.ReadString();
var playerName = _serializer.ReadString();
var additionalData = _serializer.ReadData(_serializer.Size);
_authorizationManager.OnAuthorization(peerId, key, playerName, additionalData);
return; return;
} }
var player = _authorizationManager.GetPlayer(peerId);
if (player == null) if (player == null)
{ {
_logger.Warn($"Peer not authorized {peerId} trying to {op}"); _logger.Warn($"Peer not authorized {peerId} trying to {op}");