Generate valid machine id (#245)

* Fix machine id generated by using Unity machine identifier

* EOL

* Code description

* Prepare code for unit tests

n

* EOL

* Rename interface to IMachineIdentifierProvider

* Follow the naming pattern and remove DataProvider from Id providers

Author: konraddysput

Runtime/Common/GuidHelper.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Security.Cryptography;
+using System.Text;
 
 namespace Backtrace.Unity.Extensions
 {
@@ -23,5 +25,20 @@ public static bool IsNullOrEmpty(string guid)
             const string emptyGuid = "00000000-0000-0000-0000-000000000000";
             return string.IsNullOrEmpty(guid) || guid == emptyGuid;
         }
+
+        /// <summary>
+        /// Converts a random string into a guid representation.
+        /// </summary>
+        public static Guid FromString(string value)
+        {
+            if (string.IsNullOrEmpty(value))
+            {
+                return Guid.Empty;
+            }
+            // to make sure we're supporting old version of Unity that can use .NET 3.5 
+            // we're using an older API to generate a GUID.
+            MD5 md5 = new MD5CryptoServiceProvider();
+            return new Guid(md5.ComputeHash(Encoding.UTF8.GetBytes(value)));
+        }
     }
 }

Runtime/Model/Attributes/MachineAttributeProvider.cs

@@ -1,5 +1,4 @@
 using Backtrace.Unity.Common;
-using Backtrace.Unity.Extensions;
 using System;
 using System.Collections.Generic;
 using System.Globalization;

Tests/Runtime/Client/Mocks.meta renamed to Runtime/Model/DataProvider.meta

@@ -0,0 +1,8 @@
+[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Backtrace.Unity.Tests.Runtime")]
+namespace Backtrace.Unity.Model.DataProvider
+{
+    internal interface IMachineIdentifierProvider
+    {
+        string Get();
+    }
+}

Runtime/Model/DataProvider/IMachineIdentifierProvider.cs

@@ -0,0 +1,9 @@
+[assembly: System.Runtime.CompilerServices.InternalsVisibleTo("Backtrace.Unity.Tests.Runtime")]
+namespace Backtrace.Unity.Model.DataProvider
+{
+    internal interface ISessionStorageDataProvider
+    {
+        void SetString(string key, string value);
+        string GetString(string key);
+    }
+}

Runtime/Model/DataProvider/IMachineIdentifierProvider.cs.meta

@@ -0,0 +1,35 @@
+using Backtrace.Unity.Extensions;
+using System;
+using System.Linq;
+using System.Net.NetworkInformation;
+
+namespace Backtrace.Unity.Model.DataProvider
+{
+    internal class NetworkIdentifierProvider : IMachineIdentifierProvider
+    {
+        public string Get()
+        {
+            var interfaces = NetworkInterface.GetAllNetworkInterfaces()
+               .Where(n => n.OperationalStatus == OperationalStatus.Up);
+
+            foreach (var @interface in interfaces)
+            {
+                var physicalAddress = @interface.GetPhysicalAddress();
+                if (physicalAddress == null)
+                {
+                    continue;
+                }
+                var macAddress = physicalAddress.ToString();
+                if (string.IsNullOrEmpty(macAddress))
+                {
+                    continue;
+                }
+                string hex = macAddress.Replace(":", string.Empty);
+                var value = Convert.ToInt64(hex, 16);
+                return GuidHelper.FromLong(value).ToString();
+            }
+
+            return null;
+        }
+    }
+}

Runtime/Model/DataProvider/ISessionStorageDataProvider.cs

@@ -0,0 +1,17 @@
+using UnityEngine;
+
+namespace Backtrace.Unity.Model.DataProvider
+{
+    internal class SessionStorageDataProvider : ISessionStorageDataProvider
+    {
+        public string GetString(string key)
+        {
+            return PlayerPrefs.GetString(key);
+        }
+
+        public void SetString(string key, string value)
+        {
+            PlayerPrefs.SetString(key, value);
+        }
+    }
+}