Add the named IOptions for AuthenticationTicketOptions to the DI cont…

…ainer
melittleman · Feb 25, 2024 · d4fc219 · d4fc219
1 parent c1e4a9d
commit d4fc219
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/DependencyInjection/Extensions/RedisConnectionBuilderExtensions.cs b/src/DependencyInjection/Extensions/RedisConnectionBuilderExtensions.cs
@@ -94,8 +94,16 @@ public static IRedisConnectionBuilder AddRedisTicketStore(
         ArgumentNullException.ThrowIfNull(builder);
 
         RedisAuthenticationTicketOptions ticketOptions = new();
-        configure?.Invoke(ticketOptions);
 
+        if (configure is not null)
+        {
+            builder.Services.Configure(builder.Name, configure);
+            configure.Invoke(ticketOptions);
+        }
+
+        // TODO: Should we look into 'TryAddKeyedTransient' here?
+        // Means we could theoretically support multiple different ticket store implementations.
+        // e.g. External Auth Schemes like GitHub, Microsoft etc. vs the internal Auth Scheme.
         builder.Services.TryAddTransient<ITicketStore>(s =>
         {
             IRedisConnectionProvider provider = s.GetRequiredService<IRedisConnectionProvider>();

diff --git a/src/DependencyInjection/Options/RedisAuthenticationTicketOptions.cs b/src/DependencyInjection/Options/RedisAuthenticationTicketOptions.cs
@@ -1,8 +1,10 @@
-namespace RedisKit.DependencyInjection.Options;
+using Microsoft.AspNetCore.Identity;
+
+namespace RedisKit.DependencyInjection.Options;
 
 public sealed record RedisAuthenticationTicketOptions
 {
     public string KeyPrefix { get; set; } = "auth-tickets:";
 
-    public string? CookieSchemeName { get; set; }
+    public string CookieSchemeName { get; set; } = IdentityConstants.ApplicationScheme;
 }