Skip to content

Commit 62a9d1d

Browse files
committed
fix: Register singleton settings in AddCoreServices not cover all config scenarios
1 parent 90e84b6 commit 62a9d1d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

libs/SailthruSDK.Extensions.DependencyInjection/ServiceCollectionExtensions.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ public static IServiceCollection AddSailthru(
7070
Ensure.IsNotNull(configuration, nameof(configuration));
7171

7272
services.Configure<SailthruSettings>(configuration.GetSection(SailthruSettings.ConfigurationSection));
73-
services.AddSingleton(sp => sp.GetRequiredService<IOptions<SailthruSettings>>().Value);
7473

7574
AddCoreServices(services);
7675

@@ -82,6 +81,8 @@ static void AddCoreServices(IServiceCollection services)
8281
services.AddHttpClient<SailthruClient>(
8382
"Sailthru",
8483
(sp, http) => ConfigureHttpClient(sp, http));
84+
85+
services.AddSingleton(sp => sp.GetRequiredService<IOptions<SailthruSettings>>().Value);
8586
}
8687

8788
static void ConfigureHttpClient(IServiceProvider services, HttpClient http)

0 commit comments

Comments
 (0)