Adds username validation upon logging (SubnauticaNitrox#1285)

* Added username validation upon login

* Numbers wasn't display

* Updated text

Author: dartasen

NitroxClient/MonoBehaviours/Gui/MainMenu/JoinServer.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections;
 using System.Diagnostics;
+using System.Text.RegularExpressions;
 using NitroxClient.Communication.Abstract;
 using NitroxClient.Communication.Exceptions;
 using NitroxClient.Communication.MultiplayerSession;
@@ -380,9 +381,10 @@ private void OnJoinClick()
 
             string playerName = playerNameText.text;
 
-            if (string.IsNullOrEmpty(playerName))
+            //https://regex101.com/r/eTWiEs/2/
+            if (!Regex.IsMatch(playerName, @"^[a-zA-Z0-9._-]{3,25}$"))
             {
-                NotifyUser("Please enter a player name and colour before trying to join a server");
+                NotifyUser("Please enter a valid player name !\n\n It must not contain any space or doubtful characters\n Allowed characters : A-Z a-z 0-9 _ . -\nLength : [3, 25]");
                 return;
             }
 
@@ -416,8 +418,9 @@ private void SessionConnectionStateChangedHandler(IMultiplayerSessionConnectionS
                 case MultiplayerSessionConnectionStage.ESTABLISHING_SERVER_POLICY:
                     Log.InGame("Requesting session policy information...");
                     break;
+
                 case MultiplayerSessionConnectionStage.AWAITING_RESERVATION_CREDENTIALS:
-                    Log.InGame("Waiting for User Input...");
+                    Log.InGame("Waiting for user input...");
                     RightSideMainMenu.OpenGroup("Join Server");
                     FocusPlayerNameTextbox();
                     break;
@@ -434,6 +437,7 @@ private void SessionConnectionStateChangedHandler(IMultiplayerSessionConnectionS
                     LoadingScreenVersionText.Initialize();
 
                     break;
+
                 case MultiplayerSessionConnectionStage.SESSION_RESERVATION_REJECTED:
                     Log.InGame("Reservation rejected...");
 
@@ -450,6 +454,7 @@ private void SessionConnectionStateChangedHandler(IMultiplayerSessionConnectionS
                         });
 
                     break;
+
                 case MultiplayerSessionConnectionStage.DISCONNECTED:
                     Log.Info("Disconnected from server");
                     break;
@@ -463,14 +468,12 @@ private void NotifyUser(string notificationMessage, Action continuationAction =
                 return;
             }
 
-            Action wrappedAction = () =>
+            MainMenuNotification notificationDialog = gameObject.AddComponent<MainMenuNotification>();
+            notificationDialog.ShowNotification(notificationMessage, () =>
             {
                 continuationAction?.Invoke();
                 Destroy(gameObject.GetComponent<MainMenuNotification>(), 0.0001f);
-            };
-
-            MainMenuNotification notificationDialog = gameObject.AddComponent<MainMenuNotification>();
-            notificationDialog.ShowNotification(notificationMessage, wrappedAction);
+            });
         }
 
         private void StopMultiplayerClient()

NitroxClient/MonoBehaviours/Gui/MainMenu/MainMenuNotification.cs

@@ -66,19 +66,19 @@ private GUISkin GetGUISkin(string skinName, int labelWidth)
         {
             return GUISkinUtils.RegisterDerivedOnce(skinName, s =>
             {
-                s.textField.fontSize = 14;
+                s.textField.fontSize = 20;
                 s.textField.richText = false;
-                s.textField.alignment = TextAnchor.MiddleLeft;
+                s.textField.alignment = TextAnchor.MiddleCenter;
                 s.textField.wordWrap = true;
                 s.textField.stretchHeight = true;
                 s.textField.padding = new RectOffset(10, 10, 5, 5);
 
-                s.label.fontSize = 14;
-                s.label.alignment = TextAnchor.MiddleRight;
+                s.label.fontSize = 20;
+                s.label.alignment = TextAnchor.MiddleCenter;
                 s.label.stretchHeight = true;
                 s.label.fixedWidth = labelWidth;
 
-                s.button.fontSize = 14;
+                s.button.fontSize = 16;
                 s.button.stretchHeight = true;
             });
         }

NitroxServer/Communication/Packets/Processors/MultiplayerSessionReservationRequestProcessor.cs

@@ -18,7 +18,7 @@ public MultiplayerSessionReservationRequestProcessor(PlayerManager playerManager
 
         public override void Process(MultiplayerSessionReservationRequest packet, NitroxConnection connection)
         {
-            Log.Info("Processing reservation request...");
+            Log.Info($"Processing reservation request from {packet.AuthenticationContext.Username}");
 
             string correlationId = packet.CorrelationId;
             PlayerSettings playerSettings = packet.PlayerSettings;
@@ -29,7 +29,7 @@ public override void Process(MultiplayerSessionReservationRequest packet, Nitrox
                 authenticationContext,
                 correlationId);
 
-            Log.Info($"Reservation processed successfully: {reservation}...");
+            Log.Info($"Reservation processed successfully: Username: {packet.AuthenticationContext.Username} - {reservation}");
 
             connection.SendPacket(reservation);
         }