Merge pull request #114 from marcdemz/bug/WIN-314

[Bug] WIN-314 - Environment variables should have higher priority

Author: Todd Lair

Src/StackifyLib/Config.cs

@@ -191,28 +191,6 @@ internal static string Get(string key, string defaultValue = null)
             {
                 if (string.IsNullOrWhiteSpace(key) == false)
                 {
-#if NETCORE || NETCOREX
-                    if (_configuration != null)
-                    {
-                        var appSettings = _configuration.GetSection("Stackify");
-                        v = appSettings[key.Replace("Stackify.", string.Empty)];
-
-                        if (string.IsNullOrEmpty(v))
-                        {
-                            // Search in Retrace, but key will likely still be Stackify.name, not Retrace.name in the code
-                            var retraceAppSettings = _configuration.GetSection("Retrace");
-                            v = retraceAppSettings[key.Replace("Stackify.", string.Empty)];
-                        }
-                    }
-#endif
-
-#if NETFULL
-				    if (string.IsNullOrEmpty(v))
-				    {
-				        v = System.Configuration.ConfigurationManager.AppSettings[key];
-				    }
-#endif
-
                     if (string.IsNullOrEmpty(v))
                     {
                         v = System.Environment.GetEnvironmentVariable(key);
@@ -233,6 +211,29 @@ internal static string Get(string key, string defaultValue = null)
                     {
                         v = System.Environment.GetEnvironmentVariable("RETRACE_" + key.Substring(9).Replace('.', '_').ToUpperInvariant());
                     }
+
+#if NETCORE || NETCOREX
+                    if (_configuration != null && string.IsNullOrEmpty(v))
+                    {
+                        var appSettings = _configuration.GetSection("Stackify");
+                        v = appSettings[key.Replace("Stackify.", string.Empty)];
+
+                        if (string.IsNullOrEmpty(v))
+                        {
+                            // Search in Retrace, but key will likely still be Stackify.name, not Retrace.name in the code
+                            var retraceAppSettings = _configuration.GetSection("Retrace");
+                            v = retraceAppSettings[key.Replace("Stackify.", string.Empty)];
+                        }
+                    }
+#endif
+
+#if NETFULL
+				    if (string.IsNullOrEmpty(v))
+				    {
+				        v = System.Configuration.ConfigurationManager.AppSettings[key];
+				    }
+#endif
+                    
                 }
             }
             finally