Let environment variables overwrite configuration file settings in case of ambiguity (loic-sharma#501)

jeroenlandheer · web-flow · commit d14aa5ad5c7a · 2020-05-03T14:03:47.000-07:00
Co-authored-by: Jeroen Landheer &lt;jlandheer@bintelligence.nl&gt;
diff --git a/src/BaGet/Extensions/IHostBuilderExtensions.cs b/src/BaGet/Extensions/IHostBuilderExtensions.cs
@@ -12,11 +12,10 @@ public static IHostBuilder ConfigureBaGetConfiguration(this IHostBuilder builder
         {
             return builder.ConfigureAppConfiguration((context, config) =>
             {
-                config.AddEnvironmentVariables();
-
                 config
                     .SetBasePath(Environment.CurrentDirectory)
-                    .AddJsonFile("appsettings.json", optional: true, reloadOnChange: true);
+                    .AddJsonFile("appsettings.json", optional: true, reloadOnChange: true)
+                    .AddEnvironmentVariables();
 
                 if (args != null)
                 {

Original file line number	Diff line number	Diff line change
`@@ -12,11 +12,10 @@ public static IHostBuilder ConfigureBaGetConfiguration(this IHostBuilder builder`
`12`	`12`	`{`
`13`	`13`	`return builder.ConfigureAppConfiguration((context, config) =>`
`14`	`14`	`{`
`15`		`- config.AddEnvironmentVariables();`
`16`		`-`
`17`	`15`	`config`
`18`	`16`	`.SetBasePath(Environment.CurrentDirectory)`
`19`		`- .AddJsonFile("appsettings.json", optional: true, reloadOnChange: true);`
	`17`	`+ .AddJsonFile("appsettings.json", optional: true, reloadOnChange: true)`
	`18`	`+ .AddEnvironmentVariables();`
`20`	`19`
`21`	`20`	`if (args != null)`
`22`	`21`	`{`