Merge pull request #38 from exceptionless/environment-variables

Pull request for  #37 Added the ability to read settings from environmental variables.

Author: niemyjski

Source/Extras/Extensions/ExceptionlessExtraConfigurationExtensions.cs

@@ -1,4 +1,6 @@
 using System;
+using System.Collections;
+using System.Collections.Generic;
 using System.Configuration;
 using System.Diagnostics;
 using System.IO;
@@ -15,6 +17,8 @@
 
 namespace Exceptionless {
     public static class ExceptionlessExtraConfigurationExtensions {
+        private static Dictionary<string, string> _environmentVariables;
+
         /// <summary>
         /// Reads the Exceptionless configuration from the app.config or web.config file.
         /// </summary>
@@ -68,19 +72,16 @@ public static void ReadAllConfig(this ExceptionlessConfiguration config, params
                 config.ReadFromAttributes(configAttributesAssemblies);
 
             config.ReadFromConfigSection();
+            config.ReadFromAppSettings();
+            config.ReadFromEnvironmentalVariables();
             config.ApplySavedServerSettings();
         }
 
         /// <summary>
-        /// Reads the Exceptionless configuration from the app.config or web.config file.
+        /// Reads the Exceptionless configuration from the app.config or web.config files configuration section.
         /// </summary>
         /// <param name="config">The configuration object you want to apply the attribute settings to.</param>
         public static void ReadFromConfigSection(this ExceptionlessConfiguration config) {
-            // If an appsetting is present for ApiKey, then it will override the other api keys
-            string apiKeyOverride = ConfigurationManager.AppSettings["Exceptionless:ApiKey"];
-            if (IsValidApiKey(apiKeyOverride))
-                config.ApiKey = apiKeyOverride;
-
             ExceptionlessSection section = null;
 
             try {
@@ -93,9 +94,8 @@ public static void ReadFromConfigSection(this ExceptionlessConfiguration config)
                 return;
 
             config.Enabled = section.Enabled;
-
-            // Only update if it hasn't already been set via app settings.
-            if (!IsValidApiKey(apiKeyOverride) && IsValidApiKey(section.ApiKey))
+            
+            if (IsValidApiKey(section.ApiKey))
                 config.ApiKey = section.ApiKey;
 
             if (!String.IsNullOrEmpty(section.ServerUrl))
@@ -161,6 +161,61 @@ public static void ReadFromConfigSection(this ExceptionlessConfiguration config)
             }
         }
 
+        /// <summary>
+        /// Reads the Exceptionless configuration from the app.config or web.config files app settings.
+        /// </summary>
+        /// <param name="config">The configuration object you want to apply the attribute settings to.</param>
+        public static void ReadFromAppSettings(this ExceptionlessConfiguration config) {
+            string apiKey = ConfigurationManager.AppSettings["Exceptionless:ApiKey"];
+            if (IsValidApiKey(apiKey))
+                config.ApiKey = apiKey;
+
+            bool enabled;
+            if (Boolean.TryParse(ConfigurationManager.AppSettings["Exceptionless:Enabled"], out enabled))
+                config.Enabled = enabled;
+            
+            string serverUrl = ConfigurationManager.AppSettings["Exceptionless:ServerUrl"];
+            if (!String.IsNullOrEmpty(serverUrl))
+                config.ServerUrl = serverUrl;
+        }
+
+        /// <summary>
+        /// Reads the Exceptionless configuration from Environment Variables.
+        /// </summary>
+        /// <param name="config">The configuration object you want to apply the attribute settings to.</param>
+        public static void ReadFromEnvironmentalVariables(this ExceptionlessConfiguration config) {
+            string apiKey = GetEnvironmentalVariable("Exceptionless:ApiKey");
+            if (IsValidApiKey(apiKey))
+                config.ApiKey = apiKey;
+
+            bool enabled;
+            if (Boolean.TryParse(GetEnvironmentalVariable("Exceptionless:Enabled"), out enabled))
+                config.Enabled = enabled;
+            
+            string serverUrl = GetEnvironmentalVariable("Exceptionless:ServerUrl");
+            if (!String.IsNullOrEmpty(serverUrl))
+                config.ServerUrl = serverUrl;
+        }
+
+        private static string GetEnvironmentalVariable(string name) {
+            if (String.IsNullOrEmpty(name))
+                return null;
+            
+            if (_environmentVariables == null) {
+                try {
+                    _environmentVariables = Environment.GetEnvironmentVariables().Cast<DictionaryEntry>().ToDictionary(e => e.Key.ToString(), e => e.Value.ToString());
+                } catch (Exception ex) {
+                    _environmentVariables = new Dictionary<string, string>();
+                    return null;
+                }
+            }
+            
+            if (!_environmentVariables.ContainsKey(name))
+                return null;
+
+            return _environmentVariables[name];
+        }
+
         private static bool IsValidApiKey(string apiKey) {
             return !String.IsNullOrEmpty(apiKey) && apiKey != "API_KEY_HERE";
         }

Source/Shared/Configuration/ExceptionlessAttribute.cs

@@ -32,6 +32,7 @@ public ExceptionlessAttribute(string apiKey) {
         /// </summary>
         /// ///
         /// <value><c>true</c> to enable SSL; otherwise, <c>false</c>.</value>
+        [ObsoleteAttribute("This property will be removed in a future release.")]
         public bool EnableSSL { get; set; }
 
         /// <summary>

Source/Shared/Configuration/ExceptionlessConfiguration.cs

@@ -97,6 +97,7 @@ public string ApiKey {
         /// <summary>
         /// Whether or not the client should use SSL when communicating with the server.
         /// </summary>
+        [ObsoleteAttribute("This property will be removed in a future release.")]
         public bool EnableSSL { get; set; }
 
         /// <summary>