Minor config changes

Author: TwoTenPvP

MLAPI/Data/NetworkingConfiguration.cs

@@ -10,7 +10,7 @@ namespace MLAPI
     public class NetworkingConfiguration
     {
         public ushort ProtocolVersion = 0;
-        public Dictionary<string, QosType> Channels = new Dictionary<string, QosType>();
+        public SortedDictionary<string, QosType> Channels = new SortedDictionary<string, QosType>();
         public List<string> MessageTypes = new List<string>();
         public int MessageBufferSize = 65535;
         public int MaxMessagesPerFrame = 150;
@@ -19,16 +19,16 @@ public class NetworkingConfiguration
         public string Address = "127.0.0.1";
         public int ClientConnectionBufferTimeout = 10;
         public bool ConnectionApproval = false;
-        public Action<byte[], int, Action<int, bool>> ConnectionApprovalCallback;
-        public byte[] ConnectionData;
+        public Action<byte[], int, Action<int, bool>> ConnectionApprovalCallback = null;
+        public byte[] ConnectionData = new byte[0];
         public bool HandleObjectSpawning = true;
         //TODO
         public bool CompressMessages = false;
         //Should only be used for dedicated servers and will require the servers RSA keypair being hard coded into clients in order to exchange a AES key
         //TODO
         public bool EncryptMessages = false;
-        public bool UseUPnP = true;
-        public Action<bool, IPAddress> UPnPCompleteCallback;
+        public bool UseUPnP = false;
+        public Action<bool, IPAddress> UPnPCompleteCallback = null;
 
         //Cached config hash
         private byte[] ConfigHash = null;

MLAPI/Helper/UPnPHelper.cs

@@ -9,8 +9,9 @@
 
 namespace MLAPI.Helper
 {
-    public class UPnPHelper
+    public static class UPnPHelper
     {
+        //Needs to be threaded, Currently freezes the game for up to 10 sec
         internal static void AttemptPortMap(int port, Action<bool, IPAddress> callback)
         {
             bool invoked = false;
@@ -49,7 +50,7 @@ internal static void AttemptPortMap(int port, Action<bool, IPAddress> callback)
                         Mapping[] mappings = task.Result.ToArray();
                         if(mappings.Length == 0)
                         {
-                            if (!invoked)
+                            if (!invoked && callback != null)
                                 callback(false, publicIPAddress);
                             invoked = true;
                         }
@@ -59,7 +60,7 @@ internal static void AttemptPortMap(int port, Action<bool, IPAddress> callback)
                             {
                                 if(mappings[i].PrivatePort == port)
                                 {
-                                    if (!invoked)
+                                    if (!invoked && callback != null)
                                         callback(true, publicIPAddress);
                                     invoked = true;
                                 }
@@ -76,7 +77,7 @@ internal static void AttemptPortMap(int port, Action<bool, IPAddress> callback)
             {
                 if (e.InnerException is NatDeviceNotFoundException)
                 {
-                    if (!invoked)
+                    if (!invoked && callback != null)
                         callback(false, publicIPAddress);
                     invoked = true;
                 }

MLAPI/MLAPI.csproj

@@ -12,6 +12,8 @@
     <TargetFrameworkVersion>v3.5</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <TargetFrameworkProfile />
+    <NuGetPackageImportStamp>
+    </NuGetPackageImportStamp>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
     <DebugSymbols>true</DebugSymbols>