Make sure referer header is reported with request interception (#630)

Also Headers should be a `Dictionary<string, string>`

Author: kblok

lib/PuppeteerSharp.Tests/PageTests/SetRequestInterceptionTests.cs

@@ -39,6 +39,23 @@ public async Task ShouldIntercept()
             Assert.True(response.Ok);
         }
 
+        [Fact]
+        public async Task ShouldContainRefererHeader()
+        {
+            await Page.SetRequestInterceptionAsync(true);
+            var requests = new List<Request>();
+
+            Page.Request += async (sender, e) =>
+            {
+                await e.Request.ContinueAsync();
+                requests.Add(e.Request);
+            };
+
+            await Page.GoToAsync(TestConstants.ServerUrl + "/one-style.html");
+            Assert.Contains("/one-style.css", requests[1].Url);
+            Assert.Contains("/one-style.html", requests[1].Headers["Referer"]);
+        }
+
         [Fact]
         public async Task ShouldProperlyReturnNavigationResponseWhenURLHasCookies()
         {
@@ -167,8 +184,10 @@ public async Task ShouldAmendHTTPHeaders()
             await Page.SetRequestInterceptionAsync(true);
             Page.Request += async (sender, e) =>
             {
-                var headers = new Dictionary<string, object>(e.Request.Headers);
-                headers["FOO"] = "bar";
+                var headers = new Dictionary<string, string>(e.Request.Headers)
+                {
+                    ["FOO"] = "bar"
+                };
                 await e.Request.ContinueAsync(new Payload { Headers = headers });
             };
             await Page.GoToAsync(TestConstants.EmptyPage);

lib/PuppeteerSharp/Page.cs

@@ -766,7 +766,7 @@ public Task<Response> GoToAsync(string url, int? timeout = null, WaitUntilNaviga
         /// Navigates to an url
         /// </summary>
         /// <param name="url">URL to navigate page to. The url should include scheme, e.g. https://.</param>
-        /// <param name="waitUntil">When to consider navigation succeeded.
+        /// <param name="waitUntil">When to consider navigation succeeded.</param>
         /// <returns>Task which resolves to the main resource response. In case of multiple redirects, the navigation will resolve with the response of the last redirect</returns>
         /// <seealso cref="GoToAsync(string, NavigationOptions)"/>
         public Task<Response> GoToAsync(string url, WaitUntilNavigation waitUntil)

lib/PuppeteerSharp/Payload.cs

@@ -13,14 +13,6 @@ namespace PuppeteerSharp
     /// </summary>
     public class Payload
     {
-        /// <summary>
-        /// Initializes a new instance of the <see cref="PuppeteerSharp.Payload"/> class.
-        /// </summary>
-        public Payload()
-        {
-            Headers = new Dictionary<string, object>();
-        }
-
         /// <summary>
         /// Gets or sets the HTTP method.
         /// </summary>
@@ -38,7 +30,7 @@ public Payload()
         /// </summary>
         /// <value>HTTP headers.</value>
         [JsonProperty("headers")]
-        public Dictionary<string, object> Headers { get; set; }
+        public Dictionary<string, string> Headers { get; set; } = new Dictionary<string, string>();
         /// <summary>
         /// Gets or sets the URL.
         /// </summary>

lib/PuppeteerSharp/Request.cs

@@ -57,7 +57,7 @@ internal Request(
             Frame = frame;
             RedirectChainList = redirectChain;
 
-            Headers = new Dictionary<string, object>();
+            Headers = new Dictionary<string, string>();
             foreach (var keyValue in payload.Headers)
             {
                 Headers[keyValue.Key] = keyValue.Value;
@@ -115,7 +115,7 @@ internal Request(
         /// Gets or sets the HTTP headers.
         /// </summary>
         /// <value>HTTP headers.</value>
-        public Dictionary<string, object> Headers { get; internal set; }
+        public Dictionary<string, string> Headers { get; internal set; }
         /// <summary>
         /// Gets or sets the URL.
         /// </summary>