Don't use written bytes when http compression is enabled (#3964)

This commit fixes a bug whereby written bytes captured when disable direct streaming is enabled are captured before they are compressed, so should not be used for the content of the request when http compression is enabled.

Author: russcam

src/Elasticsearch.Net/Connection/HttpConnection.cs

@@ -297,7 +297,7 @@ protected virtual HttpRequestMessage CreateRequestMessage(RequestData requestDat
 
 			if (!requestData.RunAs.IsNullOrEmpty())
 				requestMessage.Headers.Add(RequestData.RunAsSecurityHeader, requestData.RunAs);
-			
+
 			return requestMessage;
 		}
 
@@ -316,7 +316,8 @@ private static void SetContent(HttpRequestMessage message, RequestData requestDa
 				else
 					requestData.PostData.Write(stream, requestData.ConnectionSettings);
 
-				if (requestData.PostData.DisableDirectStreaming.GetValueOrDefault(false))
+				// the written bytes are uncompressed, so can only be used when http compression isn't used
+				if (requestData.PostData.DisableDirectStreaming.GetValueOrDefault(false) && !requestData.HttpCompression)
 				{
 					message.Content = new ByteArrayContent(requestData.PostData.WrittenBytes);
 					stream.Dispose();
@@ -349,7 +350,8 @@ private static async Task SetContentAsync(HttpRequestMessage message, RequestDat
 				else
 					await requestData.PostData.WriteAsync(stream, requestData.ConnectionSettings, cancellationToken).ConfigureAwait(false);
 
-				if (requestData.PostData.DisableDirectStreaming.GetValueOrDefault(false))
+				// the written bytes are uncompressed, so can only be used when http compression isn't used
+				if (requestData.PostData.DisableDirectStreaming.GetValueOrDefault(false) && !requestData.HttpCompression)
 				{
 					message.Content = new ByteArrayContent(requestData.PostData.WrittenBytes);
 					stream.Dispose();