testing and fix validation

digdir · Mar 12, 2024 · 4eb60f2 · 4eb60f2
1 parent 2ab22d2
commit 4eb60f2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/src/Digdir.Domain.Dialogporten.Infrastructure/InfrastructureExtensions.cs b/src/Digdir.Domain.Dialogporten.Infrastructure/InfrastructureExtensions.cs
@@ -26,6 +26,7 @@
 using Digdir.Domain.Dialogporten.Infrastructure.Altinn.Events;
 using Digdir.Domain.Dialogporten.Infrastructure.Altinn.OrganizationRegistry;
 using Digdir.Domain.Dialogporten.Infrastructure.Altinn.ResourceRegistry;
+using StackExchange.Redis;
 
 namespace Digdir.Domain.Dialogporten.Infrastructure;
 
@@ -59,7 +60,7 @@ public static IServiceCollection AddInfrastructure(this IServiceCollection servi
         var infrastructureSettings = infrastructureConfigurationSection.Get<InfrastructureSettings>()
         ?? throw new InvalidOperationException("Failed to get Redis settings. Infrastructure settings must not be null.");
 
-        if (infrastructureSettings.Redis.Enabled)
+        if (infrastructureSettings.Redis.Enabled == true)
         {
             services.AddStackExchangeRedisCache(options =>
             {

diff --git a/src/Digdir.Domain.Dialogporten.Infrastructure/InfrastructureSettings.cs b/src/Digdir.Domain.Dialogporten.Infrastructure/InfrastructureSettings.cs
@@ -29,7 +29,7 @@ public sealed class AltinnCdnPlatformSettings
 
 public sealed class RedisSettings
 {
-    public required bool Enabled { get; init; }
+    public required bool? Enabled { get; init; }
     public required string ConnectionString { get; init; }
 }
 
@@ -93,7 +93,7 @@ internal sealed class RedisSettingsValidator : AbstractValidator<RedisSettings>
 {
     public RedisSettingsValidator()
     {
-        RuleFor(x => x.Enabled).NotEmpty();
+        RuleFor(x => x.Enabled).Must(x => x is false or true);
         RuleFor(x => x.ConnectionString).NotEmpty();
     }
 }
diff --git a/src/Digdir.Domain.Dialogporten.WebApi/appsettings.Development.json b/src/Digdir.Domain.Dialogporten.WebApi/appsettings.Development.json
@@ -6,9 +6,9 @@
     }
   },
   "Infrastructure": {
-    "Redis":{
-      "Enabled": false,
-      "ConnectionString": "redis://localhost:6379"
+    "Redis": {
+      "Enabled": true,
+      "ConnectionString": "localhost:6379"
     },
     "DialogDbConnectionString": "TODO: Add to local secrets",
     // Settings from appsettings.json, environment variables or other configuration providers.