epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #7152 from ihptru/masterserver

Browse Source 
notify user that his server does not have forwarded ports
This commit is contained in:
Oliver Brakmann
2014-12-20 13:34:51 +01:00
parent 2509bbdc8e 57897b4916
commit 087a06b3bc
1 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
OpenRA.Mods.Common/ServerTraits/MasterServerPinger.cs
View File

@@ -72,7 +72,7 @@ namespace OpenRA.Mods.Common.Server
using (var wc = new WebClient())
{
wc.Proxy = null;
wc.DownloadData(
var masterResponse = wc.DownloadData(
server.Settings.MasterServer + url.F(
server.Settings.ExternalPort, Uri.EscapeUriString(server.Settings.Name),
(int)server.State,
@@ -87,9 +87,18 @@ namespace OpenRA.Mods.Common.Server
if (isInitialPing)
{
var masterResponseText = Encoding.UTF8.GetString(masterResponse);
isInitialPing = false;
lock (masterServerMessages)
{
masterServerMessages.Enqueue("Master server communication established.");
if (masterResponseText.Contains("[001]")) // Server does not respond code
{
Log.Write("server", masterResponseText);
masterServerMessages.Enqueue("Warning: Server ports are not forwarded.");
masterServerMessages.Enqueue("Game has not been advertised online.");
}
}
}
}
}