From 3e1dd274bbe6f2c9b67e611675f8ddb041174896 Mon Sep 17 00:00:00 2001 From: Joonas Rikkonen Date: Sat, 15 Jun 2019 19:46:29 +0300 Subject: [PATCH] (d3653c241) Fixed GameServer.UnbanPlayer passing the name to BanList in lower case even though BanList was case sensitive (preventing unbanning clients with the "unban" command if their name is not in lower case) --- Barotrauma/BarotraumaServer/Source/Networking/BanList.cs | 3 ++- Barotrauma/BarotraumaServer/Source/Networking/GameServer.cs | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Barotrauma/BarotraumaServer/Source/Networking/BanList.cs b/Barotrauma/BarotraumaServer/Source/Networking/BanList.cs index f8b1c148d..58a1aa5b5 100644 --- a/Barotrauma/BarotraumaServer/Source/Networking/BanList.cs +++ b/Barotrauma/BarotraumaServer/Source/Networking/BanList.cs @@ -184,7 +184,8 @@ namespace Barotrauma.Networking public void UnbanPlayer(string name) { - var player = bannedPlayers.Find(bp => bp.Name == name); + name = name.ToLower(); + var player = bannedPlayers.Find(bp => bp.Name.ToLower() == name); if (player == null) { DebugConsole.Log("Could not unban player \"" + name + "\". Matching player not found."); diff --git a/Barotrauma/BarotraumaServer/Source/Networking/GameServer.cs b/Barotrauma/BarotraumaServer/Source/Networking/GameServer.cs index 6dbaf7cb0..c48bb85f6 100644 --- a/Barotrauma/BarotraumaServer/Source/Networking/GameServer.cs +++ b/Barotrauma/BarotraumaServer/Source/Networking/GameServer.cs @@ -2190,7 +2190,6 @@ namespace Barotrauma.Networking public override void UnbanPlayer(string playerName, string playerIP) { - playerName = playerName.ToLowerInvariant(); if (!string.IsNullOrEmpty(playerIP)) { serverSettings.BanList.UnbanIP(playerIP);