Do not consume our public-facing HTTP request event args body

Author: RenderMichael

dotnet/src/webdriver/Remote/HttpCommandExecutor.cs

@@ -280,7 +280,7 @@ private async Task<HttpResponseInfo> MakeHttpRequest(HttpRequestInfo requestInfo
                     acceptHeader.CharSet = Utf8CharsetType;
                     requestMessage.Headers.Accept.Add(acceptHeader);
 
-                    byte[] bytes = Encoding.UTF8.GetBytes(eventArgs.RequestBody);
+                    byte[] bytes = Encoding.UTF8.GetBytes(requestInfo.RequestBody);
                     requestMessage.Content = new ByteArrayContent(bytes, 0, bytes.Length);
 
                     MediaTypeHeaderValue contentTypeHeader = new MediaTypeHeaderValue(JsonMimeType);

dotnet/src/webdriver/Remote/SendingRemoteHttpRequestEventArgs.cs

@@ -37,12 +37,12 @@ public class SendingRemoteHttpRequestEventArgs : EventArgs
         /// <param name="method">The HTTP method of the request being sent.</param>
         /// <param name="fullUrl">The full URL of the request being sent.</param>
         /// <param name="requestBody">The body of the request.</param>
-        /// <exception cref="ArgumentNullException">If <paramref name="method"/>, <paramref name="fullUrl"/>, or <paramref name="requestBody"/> are null.</exception>
-        public SendingRemoteHttpRequestEventArgs(string method, string fullUrl, string requestBody)
+        /// <exception cref="ArgumentNullException">If <paramref name="method"/>, <paramref name="fullUrl"/> are null.</exception>
+        public SendingRemoteHttpRequestEventArgs(string method, string fullUrl, string? requestBody)
         {
             this.Method = method ?? throw new ArgumentNullException(nameof(method));
             this.FullUrl = fullUrl ?? throw new ArgumentNullException(nameof(fullUrl));
-            this.RequestBody = requestBody ?? throw new ArgumentNullException(nameof(requestBody));
+            this.RequestBody = requestBody;
         }
 
         /// <summary>
@@ -58,7 +58,7 @@ public SendingRemoteHttpRequestEventArgs(string method, string fullUrl, string r
         /// <summary>
         /// Gets the body of the HTTP request as a string.
         /// </summary>
-        public string RequestBody { get; }
+        public string? RequestBody { get; }
 
         /// <summary>
         /// Gets a read-only dictionary of the headers of the HTTP request.