Add possibility to overload appsettings.json with environment variables

replaysMike · Aug 5, 2024 · b7d45c9 · b7d45c9
1 parent 8f6a659
commit b7d45c9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/Binner/Binner.Web/Program.cs b/Binner/Binner.Web/Program.cs
@@ -16,8 +16,9 @@
 WebApplicationBuilder builder;
 WebHostServiceConfiguration? config;
 try
-{
+{    // Question : Can we use Config class to reduce the risk of error with BinnerWebHostService.InitializeWebHostAsync ?
     builder = WebApplication.CreateBuilder();
+    builder.Configuration.AddEnvironmentVariables();
     config = builder.Configuration.GetSection(nameof(WebHostServiceConfiguration)).Get<WebHostServiceConfiguration>();
     if (config == null)
     {

diff --git a/Binner/Library/Binner.Common/Config.cs b/Binner/Library/Binner.Common/Config.cs
@@ -14,7 +14,8 @@ public static IConfigurationRoot GetConfiguration(string appSettingsJson, string
                 throw new FileNotFoundException($"The configuration file named '{filePath}' was not found.");
             var builder = new ConfigurationBuilder()
                 .SetBasePath(path)
-                .AddJsonFile(appSettingsJson, optional: false, reloadOnChange: true);
+                .AddJsonFile(appSettingsJson, optional: false, reloadOnChange: true)
+                .AddEnvironmentVariables();
             return builder.Build();
         }
     }