Bidi: Implement Accept insecure certs

Author: kblok

lib/PuppeteerSharp.Nunit/TestExpectations/TestExpectations.local.json

@@ -705,21 +705,6 @@
       "FAIL"
     ]
   },
-  {
-    "comment": "This is part of organizing the webdriver bidi implementation, We will remove it one by one",
-    "testIdPattern": "[acceptInsecureCerts.spec] *",
-    "platforms": [
-      "darwin",
-      "linux",
-      "win32"
-    ],
-    "parameters": [
-      "webDriverBiDi"
-    ],
-    "expectations": [
-      "FAIL"
-    ]
-  },
   {
     "comment": "This is part of organizing the webdriver bidi implementation, We will remove it one by one",
     "testIdPattern": "[injected.spec] *",

lib/PuppeteerSharp.Tests/IgnoreHttpsErrorsTests/ResponseSecurityDetailsTests.cs

@@ -16,26 +16,29 @@ public ResponseSecurityDetailsTests() : base()
             DefaultOptions.AcceptInsecureCerts = true;
         }
 
-        [Test, PuppeteerTest("acceptInsecureCerts.spec", "Response.securityDetails", "Should Work")]
+        [Test, PuppeteerTest("acceptInsecureCerts.spec", "acceptInsecureCerts Response.securityDetails", "Should Work")]
         public async Task ShouldWork()
         {
-            // Checking for the TLS socket is it is in upstreams proves to be flaky in .net framework.
+            // Checking for the TLS socket is it is in upstreams proves to be flaky in .NET Framework.
             // We don't need to test that here.
 
             var response = await Page.GoToAsync(TestConstants.HttpsPrefix + "/empty.html");
-            Assert.That(response.Status, Is.EqualTo(HttpStatusCode.OK));
-            Assert.That(response.SecurityDetails, Is.Not.Null);
+            Assert.Multiple(() =>
+            {
+                Assert.That(response.Status, Is.EqualTo(HttpStatusCode.OK));
+                Assert.That(response.SecurityDetails, Is.Not.Null);
+            });
             Assert.That(response.SecurityDetails.Protocol, Does.Contain("TLS"));
         }
 
-        [Test, PuppeteerTest("acceptInsecureCerts.spec", "Response.securityDetails", "should be |null| for non-secure requests")]
+        [Test, PuppeteerTest("acceptInsecureCerts.spec", "acceptInsecureCerts Response.securityDetails", "should be |null| for non-secure requests")]
         public async Task ShouldBeNullForNonSecureRequests()
         {
             var response = await Page.GoToAsync(TestConstants.EmptyPage);
             Assert.That(response.SecurityDetails, Is.Null);
         }
 
-        [Test, PuppeteerTest("acceptInsecureCerts.spec", "Response.securityDetails", "Network redirects should report SecurityDetails")]
+        [Test, PuppeteerTest("acceptInsecureCerts.spec", "acceptInsecureCerts Response.securityDetails", "Network redirects should report SecurityDetails")]
         [Ignore("This is super flaky")]
         public async Task NetworkRedirectsShouldReportSecurityDetails()
         {
@@ -55,10 +58,13 @@ await Task.WhenAll(
 
             var response = responseTask.Result;
 
-            Assert.That(responses, Has.Count.EqualTo(2));
-            Assert.That(responses[0].Status, Is.EqualTo(HttpStatusCode.Found));
-            Assert.That(TestUtils.CurateProtocol(response.SecurityDetails.Protocol),
-                Is.EqualTo(TestUtils.CurateProtocol(requestTask.Result.ToString())));
+            Assert.Multiple(() =>
+            {
+                Assert.That(responses, Has.Count.EqualTo(2));
+                Assert.That(responses[0].Status, Is.EqualTo(HttpStatusCode.Found));
+                Assert.That(TestUtils.CurateProtocol(response.SecurityDetails.Protocol),
+                    Is.EqualTo(TestUtils.CurateProtocol(requestTask.Result.ToString())));
+            });
         }
     }
 }

lib/PuppeteerSharp/Bidi/BidiHttpResponse.cs

@@ -39,6 +39,10 @@ private BidiHttpResponse(WebDriverBiDi.Network.ResponseData data, BidiHttpReques
         Status = (HttpStatusCode)data.Status;
         Url = data.Url;
         _fromCache = data.FromCache;
+
+        // TODO: Implement SecurityDetails support when webdriverbidi-net library supports extensibility
+        // The upstream puppeteer implementation accesses a non-standard 'goog:securityDetails' property
+        // which is not yet exposed in the webdriverbidi-net library
     }
 
     // Internal constructor for synthetic responses (e.g., cached history navigation)