dont rely on a separate flag

Author: DeagleGross

scenarios/rejection.benchmarks.yml

@@ -100,7 +100,6 @@ scenarios:
       variables:
         httpSysUrlPrefix: "http://testserver:{{serverPort}}"
         serverScheme: http
-        httpOnly: true
     load:
       job: wrk
       variables:
@@ -156,7 +155,6 @@ scenarios:
       variables:
         enableHostHeaderValidation: true
         serverScheme: http
-        httpOnly: true
     load:
       job: wrk
       variables:

src/BenchmarksApps/TLS/HttpSys/Program.cs

@@ -14,12 +14,14 @@
 var certPublicKeyLength = certPublicKeySpecified ? certPublicKeyConfig : 2048;
 var urlPrefix = builder.Configuration["httpSysUrlPrefix"];
 
-// for investigation purposes you can disable https, but the point of TLS apps is to measure TLS scenarios
-var httpOnly = bool.TryParse(builder.Configuration["httpOnly"], out var httpOnlyConfig) && httpOnlyConfig;
-
 // endpoints
 var listeningEndpoints = builder.Configuration["urls"] ?? "https://localhost:5000/";
 var httpsIpPort = listeningEndpoints.Split(";").FirstOrDefault(x => x.Contains("https"))?.Replace("https://", "");
+var httpOnly = httpsIpPort is null; // in case TLS is disabled. Only for debug purposes - this app is designed to measure TLS scenario
+if (httpOnly)
+{
+    Console.WriteLine("[Note] Server scheme is HTTP, not HTTPS.");
+}
 
 // debug
 var writeCertValidationEventsToConsole = bool.TryParse(builder.Configuration["certValidationConsoleEnabled"], out var certValidationConsoleEnabled) && certValidationConsoleEnabled;

src/BenchmarksApps/TLS/Kestrel/Program.cs

@@ -21,13 +21,25 @@
 var certPublicKeySpecified = int.TryParse(builder.Configuration["certPublicKeyLength"], out var certPublicKeyConfig);
 var certPublicKeyLength = certPublicKeySpecified ? certPublicKeyConfig : 2048;
 var enableHostHeaderValidation = bool.TryParse(builder.Configuration["enableHostHeaderValidation"], out var enableHostHeaderValidationConfig) && enableHostHeaderValidationConfig;
-
-// for investigation purposes you can disable https, but the point of TLS apps is to measure TLS scenarios
-var httpOnly = bool.TryParse(builder.Configuration["httpOnly"], out var httpOnlyConfig) && httpOnlyConfig;
+var supportedTlsVersions = ParseSslProtocols(builder.Configuration["tlsProtocols"]);
 
 // endpoints
 var listeningEndpoints = builder.Configuration["urls"] ?? "https://localhost:5000/";
-var supportedTlsVersions = ParseSslProtocols(builder.Configuration["tlsProtocols"]);
+
+// determine if listening is expected only on HTTP scheme
+var httpOnly = true;
+foreach (var endpoint in listeningEndpoints.Split([';'], StringSplitOptions.RemoveEmptyEntries))
+{
+    var urlPrefix = UrlPrefix.Create(endpoint);
+    if (urlPrefix.Scheme == "https")
+    {
+        httpOnly = false;
+    }
+}
+if (httpOnly)
+{
+    Console.WriteLine("[Note] Server scheme is HTTP, not HTTPS.");
+}
 
 // debug
 var writeCertValidationEventsToConsole = bool.TryParse(builder.Configuration["certValidationConsoleEnabled"], out var certValidationConsoleEnabled) && certValidationConsoleEnabled;