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

more checks for NRE and setting to disable NAT discovery

Browse Source
This commit is contained in:
Matthias Mailänder
2013-04-09 13:03:37 +02:00
parent 1603485b33
commit 3abe58c4ae
2 changed files with 49 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

75
OpenRA.Game/Game.cs
View File

@@ -265,19 +265,27 @@ namespace OpenRA
Log.AddChannel("sync", "syncreport.log"); Log.AddChannel("sync", "syncreport.log");
Log.AddChannel("server", "server.log"); Log.AddChannel("server", "server.log");
try if (Settings.Server.DiscoverNatDevices)
{ {
NatUtility.Logger = Log.Channels["server"].Writer; try
NatUtility.Verbose = Settings.Server.VerboseNatDiscovery; {
NatUtility.DeviceFound += DeviceFound; NatUtility.Logger = Log.Channels["server"].Writer;
NatUtility.DeviceLost += DeviceLost; NatUtility.Verbose = Settings.Server.VerboseNatDiscovery;
Settings.Server.NatDeviceAvailable = false; NatUtility.DeviceFound += DeviceFound;
NatUtility.StartDiscovery(); NatUtility.DeviceLost += DeviceLost;
Log.Write("server", "NAT discovery started."); Settings.Server.NatDeviceAvailable = false;
NatUtility.StartDiscovery();
Log.Write("server", "NAT discovery started.");
}
catch (Exception e)
{
Log.Write("server", "Can't discover UPnP-enabled device: {0}", e);
Settings.Server.NatDeviceAvailable = false;
Settings.Server.AllowUPnP = false;
}
} }
catch (Exception e) else
{ {
Log.Write("server", "Can't discover UPnP-enabled device: {0}", e);
Settings.Server.NatDeviceAvailable = false; Settings.Server.NatDeviceAvailable = false;
Settings.Server.AllowUPnP = false; Settings.Server.AllowUPnP = false;
} }
@@ -293,32 +301,38 @@ namespace OpenRA
Sound.Create(Settings.Sound.Engine); Sound.Create(Settings.Sound.Engine);
InitializeWithMods(Settings.Game.Mods); InitializeWithMods(Settings.Game.Mods);
RunAfterDelay(Settings.Server.NatDiscoveryTimeout, () => if (Settings.Server.DiscoverNatDevices)
{ {
Log.Write("server", "Stopping NAT discovery."); RunAfterDelay(Settings.Server.NatDiscoveryTimeout, () =>
{
Log.Write("server", "Stopping NAT discovery.");
try try
{ {
NatUtility.StopDiscovery(); NatUtility.StopDiscovery();
} }
catch (Exception e) catch (Exception e)
{ {
Log.Write("server", "Failed to stop NAT device discovery: {0}", e); Log.Write("server", "Failed to stop NAT device discovery: {0}", e);
Settings.Server.NatDeviceAvailable = false; Settings.Server.NatDeviceAvailable = false;
Settings.Server.AllowUPnP = false; Settings.Server.AllowUPnP = false;
} }
if (natDevice == null) if (natDevice == null)
{ {
Log.Write("server", "No NAT devices with UPnP enabled found within {0} ms deadline. Disabling automatic port forwarding.".F(Settings.Server.NatDiscoveryTimeout)); Log.Write("server", "No NAT devices with UPnP enabled found within {0} ms deadline. Disabling automatic port forwarding.".F(Settings.Server.NatDiscoveryTimeout));
Settings.Server.NatDeviceAvailable = false; Settings.Server.NatDeviceAvailable = false;
Settings.Server.AllowUPnP = false; Settings.Server.AllowUPnP = false;
} }
}); });
}
} }
public static void DeviceFound(object sender, DeviceEventArgs args) public static void DeviceFound(object sender, DeviceEventArgs args)
{ {
if (args.Device == null)
return;
Log.Write("server", "NAT device discovered."); Log.Write("server", "NAT device discovered.");
Settings.Server.NatDeviceAvailable = true; Settings.Server.NatDeviceAvailable = true;
@@ -347,6 +361,9 @@ namespace OpenRA
{ {
Log.Write("server", "NAT device lost."); Log.Write("server", "NAT device lost.");
if (args.Device == null)
return;
try try
{ {
natDevice = args.Device; natDevice = args.Device;

4
OpenRA.Game/GameRules/Settings.cs
View File

@@ -29,7 +29,8 @@ namespace OpenRA.GameRules
public int ExternalPort = 1234; public int ExternalPort = 1234;
public bool AdvertiseOnline = true; public bool AdvertiseOnline = true;
public string MasterServer = "http://master.open-ra.org/"; public string MasterServer = "http://master.open-ra.org/";
public bool AllowUPnP = true; // let the user disable it public bool DiscoverNatDevices = true; // Allow users to disable NAT discovery if problems occur
public bool AllowUPnP = true; // let the user disable it even if compatible devices are found
public bool NatDeviceAvailable = false; // internal check if discovery succeeded public bool NatDeviceAvailable = false; // internal check if discovery succeeded
public int NatDiscoveryTimeout = 1000; // ms to search for UPnP enabled NATs public int NatDiscoveryTimeout = 1000; // ms to search for UPnP enabled NATs
public bool VerboseNatDiscovery = false; // print very detailed logs for debugging public bool VerboseNatDiscovery = false; // print very detailed logs for debugging
@@ -51,6 +52,7 @@ namespace OpenRA.GameRules
ExternalPort = other.ExternalPort; ExternalPort = other.ExternalPort;
AdvertiseOnline = other.AdvertiseOnline; AdvertiseOnline = other.AdvertiseOnline;
MasterServer = other.MasterServer; MasterServer = other.MasterServer;
DiscoverNatDevices = other.DiscoverNatDevices;
AllowUPnP = other.AllowUPnP; AllowUPnP = other.AllowUPnP;
NatDeviceAvailable = other.NatDeviceAvailable; NatDeviceAvailable = other.NatDeviceAvailable;
NatDiscoveryTimeout = other.NatDiscoveryTimeout; NatDiscoveryTimeout = other.NatDiscoveryTimeout;