Fixes an obscure bug that was discovered by @TWM617 - if a configurat…

…ion section is missing (such as newly introduced integrations) it should not be defaulted to true. The new configuration validation feature revealed this problem as it validates enabled integration apis for proper settings on startup.
replaysMike · May 13, 2023 · 1e45c08 · 1e45c08 · TWM617 · May 13, 2023
1 parent 0778401
commit 1e45c08
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/Binner/Binner.Web/appsettings.json b/Binner/Binner.Web/appsettings.json
@@ -78,7 +78,7 @@
         "ApiUrl": "https://api.mouser.com"
       },
       "Arrow": {
-        "Enabled": false,
+        "Enabled": true,
         // Arrow API Keys can be obtained at https://developers.arrow.com/api/index.php/site/page?view=requestAPIKey
         "ApiKey": "",
         "Username": "",

diff --git a/Binner/Library/Binner.Model/Configuration/Integrations/ArrowConfiguration.cs b/Binner/Library/Binner.Model/Configuration/Integrations/ArrowConfiguration.cs
@@ -5,7 +5,7 @@
     /// </summary>
     public class ArrowConfiguration : IApiConfiguration
     {
-        public bool Enabled { get; set; } = true;
+        public bool Enabled { get; set; } = false;
 
         /// <summary>
         /// Your arrow.com account username/login

diff --git a/Binner/Library/Binner.Model/Configuration/Integrations/DigikeyConfiguration.cs b/Binner/Library/Binner.Model/Configuration/Integrations/DigikeyConfiguration.cs
@@ -5,7 +5,7 @@
     /// </summary>
     public class DigikeyConfiguration : IApiConfiguration
     {
-        public bool Enabled { get; set; } = true;
+        public bool Enabled { get; set; } = false;
 
         public string? ApiKey => ClientId;
 

diff --git a/Binner/Library/Binner.Model/Configuration/Integrations/MouserConfiguration.cs b/Binner/Library/Binner.Model/Configuration/Integrations/MouserConfiguration.cs
@@ -5,7 +5,7 @@
     /// </summary>
     public class MouserConfiguration : IApiConfiguration
     {
-        public bool Enabled { get; set; } = true;
+        public bool Enabled { get; set; } = false;
 
         public string? ApiKey => ApiKeys.SearchApiKey;
 

diff --git a/Binner/Library/Binner.Model/Configuration/Integrations/SwarmConfiguration.cs b/Binner/Library/Binner.Model/Configuration/Integrations/SwarmConfiguration.cs
@@ -5,7 +5,7 @@
     /// </summary>
     public class SwarmConfiguration : IApiConfiguration
     {
-        public bool Enabled { get; set; } = true;
+        public bool Enabled { get; set; } = false;
 
         /// <summary>
         /// The optional api key