chore: update the user agent string for twilio-csharp

src/Twilio/Clients/TwilioRestClient.cs

@@ -40,6 +40,11 @@ public class TwilioRestClient : ITwilioRestClient
         /// </summary>
         public string Edge { get; set; }
 
+        /// <summary>
+        /// Additions to the user agent string
+        /// </summary>
+        public string[] UserAgentExtensions { get; set; }
+
         /// <summary>
         /// Log level for logging
         /// </summary>
@@ -57,13 +62,15 @@ public class TwilioRestClient : ITwilioRestClient
         /// <param name="region">region to make requests for</param>
         /// <param name="httpClient">http client used to make the requests</param>
         /// <param name="edge">edge to make requests for</param>
+        /// <param name="userAgentExtensions">Additions to the user agent string</param>
         public TwilioRestClient(
             string username,
             string password,
             string accountSid = null,
             string region = null,
             HttpClient httpClient = null,
-            string edge = null
+            string edge = null,
+            string[] userAgentExtensions = null
         )
         {
             _username = username;
@@ -74,6 +81,7 @@ public TwilioRestClient(
 
             Region = region;
             Edge = edge;
+            UserAgentExtensions = userAgentExtensions;
         }
 
         /// <summary>
@@ -95,6 +103,9 @@ public Response Request(Request request)
             if (Edge != null)
                 request.Edge = Edge;
 
+            if (UserAgentExtensions != null)
+                request.UserAgentExtensions = UserAgentExtensions;
+
             Response response;
             try
             {
@@ -132,6 +143,9 @@ public async Task<Response> RequestAsync(Request request)
             if (Edge != null)
                 request.Edge = Edge;
 
+            if (UserAgentExtensions != null)
+                request.UserAgentExtensions = UserAgentExtensions;
+
             Response response;
             try
             {

src/Twilio/Http/Net35/WebRequestClient.cs

@@ -9,7 +9,7 @@ namespace Twilio.Http.Net35
     /// </summary>
     public class WebRequestClient : HttpClient
     {
-        private const string PlatVersion = " (.NET 3.5)";
+        private const string PlatVersion = ".NET 3.5";
         private HttpWebRequestFactory factory;
 
         public WebRequestClient(HttpWebRequestFactory factory = null)
@@ -63,7 +63,24 @@ private IHttpWebRequestWrapper BuildHttpRequest(Request request)
         {
             IHttpWebRequestWrapper httpRequest = this.factory.Create(request.ConstructUrl());
 
-            httpRequest.UserAgent = "twilio-csharp/" + AssemblyInfomation.AssemblyInformationalVersion + PlatVersion;
+            int lastSpaceIndex = PlatVersion.LastIndexOf(" ");
+            System.Text.StringBuilder PlatVersionSb= new System.Text.StringBuilder(PlatVersion);
+            PlatVersionSb[lastSpaceIndex] = '/';
+
+            string helperLibVersion = AssemblyInfomation.AssemblyInformationalVersion;
+            string osName = System.Environment.OSVersion.Platform.ToString();
+            string osArch = System.Environment.GetEnvironmentVariable("PROCESSOR_ARCHITECTURE").ToString();
+            var libraryVersion = System.String.Format("twilio-csharp/{0} ({1} {2}) {3}", helperLibVersion,osName,osArch,PlatVersionSb);
+
+            if (request.UserAgentExtensions != null)
+            {
+                foreach (var extension in request.UserAgentExtensions)
+                {
+                    libraryVersion += " " + extension;
+                }
+            }
+
+            httpRequest.UserAgent = libraryVersion;
 
             httpRequest.Method = request.Method.ToString();
             httpRequest.Accept = "application/json";

src/Twilio/Http/Request.cs

@@ -46,6 +46,11 @@ public class Request
         /// </summary>
         public string Edge { get; set; }
 
+        /// <summary>
+        /// Additions to the user agent string
+        /// </summary>
+        public string[] UserAgentExtensions { get; set; }
+
         /// <summary>
         /// Query params
         /// </summary>
@@ -86,6 +91,8 @@ public Request(HttpMethod method, string url)
         /// <param name="postParams">Post data</param>
         /// <param name="edge">Twilio edge</param>
         /// <param name="headerParams">Custom header data</param>
+        /// <param name="userAgentExtensions">Additions to the user agent string</param>
+
         public Request(
             HttpMethod method,
             Domain domain,
@@ -94,13 +101,15 @@ public Request(
             List<KeyValuePair<string, string>> queryParams = null,
             List<KeyValuePair<string, string>> postParams = null,
             string edge = null,
-            List<KeyValuePair<string, string>> headerParams = null
+            List<KeyValuePair<string, string>> headerParams = null,
+            string[] userAgentExtensions = null
         )
         {
             Method = method;
             Uri = new Uri("https://" + domain + ".twilio.com" + uri);
             Region = region;
             Edge = edge;
+            UserAgentExtensions = userAgentExtensions;
 
             QueryParams = queryParams ?? new List<KeyValuePair<string, string>>();
             PostParams = postParams ?? new List<KeyValuePair<string, string>>();

src/Twilio/Http/SystemNetHttpClient.cs

@@ -14,9 +14,9 @@ namespace Twilio.Http
     public class SystemNetHttpClient : HttpClient
     {
 #if NET451
-        private string PlatVersion = " (.NET Framework 4.5.1+)";
+        private string PlatVersion = ".NET Framework 4.5.1+";
 #else
-        private string PlatVersion = $" ({RuntimeInformation.FrameworkDescription})";
+        private string PlatVersion = $"{RuntimeInformation.FrameworkDescription}";
 #endif
 
         private readonly System.Net.Http.HttpClient _httpClient;
@@ -91,7 +91,45 @@ private HttpRequestMessage BuildHttpRequest(Request request)
             httpRequest.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue("application/json"));
             httpRequest.Headers.AcceptEncoding.Add(new StringWithQualityHeaderValue("utf-8"));
 
-            var libraryVersion = "twilio-csharp/" + AssemblyInfomation.AssemblyInformationalVersion + PlatVersion;
+            int lastSpaceIndex = PlatVersion.LastIndexOf(" ");
+            System.Text.StringBuilder PlatVersionSb= new System.Text.StringBuilder(PlatVersion);
+            PlatVersionSb[lastSpaceIndex] = '/';
+
+            string helperLibVersion = AssemblyInfomation.AssemblyInformationalVersion;
+
+            string osName = "Unknown";
+#if !NETSTANDARD1_4
+            osName = Environment.OSVersion.Platform.ToString();
+#else       
+            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+            {
+                osName = "Windows";
+            }
+            else if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
+            {
+                osName = "MacOS";
+            }
+            else if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
+            {
+                osName = "Linux";
+            }
+#endif
+            string osArch = "Unknown";
+#if !NET451
+            osArch = RuntimeInformation.OSArchitecture.ToString();
+#else
+            osArch = Environment.GetEnvironmentVariable("PROCESSOR_ARCHITECTURE").ToString(); 
+#endif
+            var libraryVersion = String.Format("twilio-csharp/{0} ({1} {2}) {3}", helperLibVersion,osName,osArch,PlatVersionSb);
+
+            if (request.UserAgentExtensions != null)
+            {
+                foreach (var extension in request.UserAgentExtensions)
+                {
+                    libraryVersion += " " + extension;
+                }
+            }
+
             httpRequest.Headers.TryAddWithoutValidation("User-Agent", libraryVersion);
 
             foreach (var header in request.HeaderParams)

test/Twilio.Test/Http/SystemNetHttpClientTest.cs

@@ -5,6 +5,7 @@
 using System.Net.Http;
 using System.Threading;
 using System.Threading.Tasks;
+using System.Text.RegularExpressions;
 using NUnit.Framework;
 using Twilio.Http;
 using HttpMethod = Twilio.Http.HttpMethod;
@@ -237,7 +238,29 @@ public void TestMakeRequestAddsHeadersAndUserAgent()
             Assert.AreEqual("utf-8", internalRequest.Headers.AcceptEncoding.ToString());
 
             Assert.IsNotNull(internalRequest.Headers.UserAgent);
-            StringAssert.StartsWith("twilio-csharp/", internalRequest.Headers.UserAgent.ToString());
+            Regex rgx = new Regex(@"^twilio-csharp/[0-9.]+\s\(\w+\s\w+\)\s[.\s\w]+/[^\s]+$");
+            Assert.IsTrue(rgx.IsMatch(internalRequest.Headers.UserAgent.ToString()));
+        }
+
+        [Test]
+        public void TestMakeRequestAddUserAgentExtensions()
+        {
+            string[] userAgentExtensions = new string[] { "twilio-run/2.0.0-test", "flex-plugin/3.4.0" };
+
+            this._mockHttp.Respond("https://api.twilio.com/v1/Resource.json", HttpStatusCode.OK);
+
+            Request testRequest = new Request(HttpMethod.Get, "https://api.twilio.com/v1/Resource.json");
+            testRequest.UserAgentExtensions = userAgentExtensions;
+            testRequest.SetAuth("username", "password");
+
+            this.TwilioHttpClient.MakeRequest(testRequest);
+
+            HttpRequestMessage internalRequest = this._mockHttp.InternalRequest;
+            string userAgent = internalRequest.Headers.UserAgent.ToString();
+            var actualUserAgent = userAgent.Split(' ');
+            var actualUserAgentExtensions = new string[userAgentExtensions.Length];
+            Array.Copy(actualUserAgent, actualUserAgent.Length - userAgentExtensions.Length, actualUserAgentExtensions, 0, userAgentExtensions.Length);
+            CollectionAssert.AreEqual(userAgentExtensions, actualUserAgentExtensions);
         }
     }
 }