Remove HttpDefaultConnectionLimit since the setting doesn't make sens…

…e anymore with modern HTTP Client usage. We have decided against adding a log warning since generally it is OK to have settings in app.config that are no longer relevant.
Particular · Mar 19, 2024 · d9cf8e7 · d9cf8e7
1 parent 0fe1565
commit d9cf8e7
Show file tree

Hide file tree

Showing 10 changed files with 0 additions and 16 deletions.
diff --git a/src/ServiceControl.AcceptanceTests/TestSupport/ServiceControlComponentRunner.cs b/src/ServiceControl.AcceptanceTests/TestSupport/ServiceControlComponentRunner.cs
@@ -53,7 +53,6 @@ async Task InitializeServiceControl(ScenarioContext context)
                 ProcessRetryBatchesFrequency = TimeSpan.FromSeconds(2),
                 TimeToRestartErrorIngestionAfterFailure = TimeSpan.FromSeconds(2),
                 MaximumConcurrencyLevel = 2,
-                HttpDefaultConnectionLimit = int.MaxValue,
                 DisableHealthChecks = true,
                 MessageFilter = messageContext =>
                 {

diff --git a/src/ServiceControl.Audit.AcceptanceTests/TestSupport/ServiceControlComponentRunner.cs b/src/ServiceControl.Audit.AcceptanceTests/TestSupport/ServiceControlComponentRunner.cs
@@ -42,7 +42,6 @@ async Task InitializeServiceControl(ScenarioContext context)
             {
                 TransportConnectionString = transportToUse.ConnectionString,
                 MaximumConcurrencyLevel = 2,
-                HttpDefaultConnectionLimit = int.MaxValue,
                 ServiceControlQueueAddress = "SHOULDNOTBEUSED",
                 MessageFilter = messageContext =>
                 {

diff --git a/...iceControl.Audit.UnitTests/ApprovalFiles/APIApprovals.PlatformSampleSettings.approved.txt b/...iceControl.Audit.UnitTests/ApprovalFiles/APIApprovals.PlatformSampleSettings.approved.txt
@@ -18,7 +18,6 @@
   "AuditRetentionPeriod": "30.00:00:00",
   "MaxBodySizeToStore": 102400,
   "ServiceName": "Particular.ServiceControl.Audit",
-  "HttpDefaultConnectionLimit": 100,
   "TransportConnectionString": null,
   "MaximumConcurrencyLevel": 32,
   "DataSpaceRemainingThreshold": 20,

diff --git a/src/ServiceControl.Audit/HostApplicationBuilderExtensions.cs b/src/ServiceControl.Audit/HostApplicationBuilderExtensions.cs
@@ -41,9 +41,6 @@ static class HostApplicationBuilderExtensions
             configuration.License(settings.LicenseFileText);
         }
 
-        // .NET default limit is 10. RavenDB in conjunction with transports that use HTTP exceeds that limit.
-        ServicePointManager.DefaultConnectionLimit = settings.HttpDefaultConnectionLimit;
-
         var transportSettings = MapSettings(settings);
         var transportCustomization = settings.LoadTransportCustomization();
 

diff --git a/src/ServiceControl.Audit/Infrastructure/Settings/Settings.cs b/src/ServiceControl.Audit/Infrastructure/Settings/Settings.cs
@@ -35,7 +35,6 @@ public Settings(string serviceName, string transportType = null, string persiste
             AuditRetentionPeriod = GetAuditRetentionPeriod();
             Port = SettingsReader.Read(SettingsRootNamespace, "Port", 44444);
             MaximumConcurrencyLevel = SettingsReader.Read(SettingsRootNamespace, "MaximumConcurrencyLevel", 32);
-            HttpDefaultConnectionLimit = SettingsReader.Read(SettingsRootNamespace, "HttpDefaultConnectionLimit", 100);
             DataSpaceRemainingThreshold = GetDataSpaceRemainingThreshold();
             ServiceControlQueueAddress = SettingsReader.Read<string>(SettingsRootNamespace, "ServiceControlQueueAddress");
             TimeToRestartAuditIngestionAfterFailure = GetTimeToRestartAuditIngestionAfterFailure();
@@ -131,7 +130,6 @@ public int MaxBodySizeToStore
 
         public string ServiceName { get; }
 
-        public int HttpDefaultConnectionLimit { get; set; }
         public string TransportConnectionString { get; set; }
         public int MaximumConcurrencyLevel { get; set; }
         public int DataSpaceRemainingThreshold { get; set; }

diff --git a/src/ServiceControl.Audit/Infrastructure/WebApi/RootController.cs b/src/ServiceControl.Audit/Infrastructure/WebApi/RootController.cs
@@ -63,7 +63,6 @@ public OkObjectResult Config()
                 PerformanceTunning = new
                 {
                     settings.MaxBodySizeToStore,
-                    settings.HttpDefaultConnectionLimit,
                 },
                 Transport = new
                 {

diff --git a/src/ServiceControl.UnitTests/ApprovalFiles/APIApprovals.PlatformSampleSettings.approved.txt b/src/ServiceControl.UnitTests/ApprovalFiles/APIApprovals.PlatformSampleSettings.approved.txt
@@ -30,7 +30,6 @@
   "ErrorRetentionPeriod": "10.00:00:00",
   "EventsRetentionPeriod": "14.00:00:00",
   "ServiceName": "Particular.ServiceControl",
-  "HttpDefaultConnectionLimit": 100,
   "TransportConnectionString": null,
   "ProcessRetryBatchesFrequency": "00:00:30",
   "TimeToRestartErrorIngestionAfterFailure": "00:01:00",

diff --git a/src/ServiceControl/HostApplicationBuilderExtensions.cs b/src/ServiceControl/HostApplicationBuilderExtensions.cs
@@ -48,9 +48,6 @@ public static void AddServiceControl(this IHostApplicationBuilder hostBuilder, S
                 EventSourceCreator.Create();
             }
 
-            // .NET default limit is 10. RavenDB in conjunction with transports that use HTTP exceeds that limit.
-            ServicePointManager.DefaultConnectionLimit = settings.HttpDefaultConnectionLimit;
-
             var transportCustomization = settings.LoadTransportCustomization();
             var transportSettings = MapSettings(settings);
 

diff --git a/src/ServiceControl/Infrastructure/Settings/Settings.cs b/src/ServiceControl/Infrastructure/Settings/Settings.cs
@@ -49,7 +49,6 @@ public class Settings
             ProcessRetryBatchesFrequency = TimeSpan.FromSeconds(30);
             MaximumConcurrencyLevel = SettingsReader.Read(SettingsRootNamespace, "MaximumConcurrencyLevel", 10);
             RetryHistoryDepth = SettingsReader.Read(SettingsRootNamespace, "RetryHistoryDepth", 10);
-            HttpDefaultConnectionLimit = SettingsReader.Read(SettingsRootNamespace, "HttpDefaultConnectionLimit", 100);
             AllowMessageEditing = SettingsReader.Read<bool>(SettingsRootNamespace, "AllowMessageEditing");
             NotificationsFilter = SettingsReader.Read<string>(SettingsRootNamespace, "NotificationsFilter");
             RemoteInstances = GetRemoteInstances().ToArray();
@@ -156,7 +155,6 @@ public TimeSpan HeartbeatGracePeriod
         public TimeSpan EventsRetentionPeriod { get; }
         public string ServiceName { get; }
 
-        public int HttpDefaultConnectionLimit { get; set; }
         public string TransportConnectionString { get; set; }
         public TimeSpan ProcessRetryBatchesFrequency { get; set; }
         public TimeSpan TimeToRestartErrorIngestionAfterFailure { get; set; }

diff --git a/src/ServiceControl/Infrastructure/WebApi/RootController.cs b/src/ServiceControl/Infrastructure/WebApi/RootController.cs
@@ -78,7 +78,6 @@ public object Config()
                 },
                 PerformanceTunning = new
                 {
-                    settings.HttpDefaultConnectionLimit,
                     settings.ExternalIntegrationsDispatchingBatchSize
                 },
                 PersistenceSettings = settings.PersisterSpecificSettings,