Remove reflection from KestrelServer constructor (#50272)

Author: halter73

src/Servers/Kestrel/Core/src/KestrelServer.cs

@@ -34,7 +34,7 @@ public KestrelServer(IOptions<KestrelServerOptions> options, IConnectionListener
             options,
             new[] { transportFactory ?? throw new ArgumentNullException(nameof(transportFactory)) },
             Array.Empty<IMultiplexedConnectionListenerFactory>(),
-            new SimpleHttpsConfigurationService(loggerFactory),
+            new SimpleHttpsConfigurationService(),
             loggerFactory,
             new KestrelMetrics(new DummyMeterFactory()));
     }
@@ -77,13 +77,6 @@ public void Dispose() { }
 
     private sealed class SimpleHttpsConfigurationService : IHttpsConfigurationService
     {
-        private readonly ILogger<HttpsConnectionMiddleware> _httpsLogger;
-
-        public SimpleHttpsConfigurationService(ILoggerFactory loggerFactory)
-        {
-            _httpsLogger = loggerFactory.CreateLogger<HttpsConnectionMiddleware>();
-        }
-
         public bool IsInitialized => true;
 
         public void Initialize(IHostEnvironment hostEnvironment, ILogger<KestrelServer> serverLogger, ILogger<HttpsConnectionMiddleware> httpsLogger)
@@ -93,7 +86,7 @@ public void Initialize(IHostEnvironment hostEnvironment, ILogger<KestrelServer>
 
         public void PopulateMultiplexedTransportFeatures(FeatureCollection features, ListenOptions listenOptions)
         {
-            HttpsConfigurationService.PopulateMultiplexedTransportFeaturesWorker(features, listenOptions, _httpsLogger);
+            throw new NotImplementedException(); // Not actually required by this impl, which never provides an IMultiplexedConnectionListenerFactory
         }
 
         public ListenOptions UseHttpsWithDefaults(ListenOptions listenOptions)