[Backport 7.7] Add RequestParams overload without optional cont… (#4660)

This commit adds back a RequestParams overload on NamespacedClientProxy
in Elasticsearch.Net that does not accept a content type
optional parameter, and removes the optional part
of the parameter on the other overload, for backwards
binary compatibility.

Co-authored-by: Russ Cam <russ.cam@elastic.co>

Author: github-actions[bot]

src/Elasticsearch.Net/ElasticLowLevelClient.cs

@@ -75,10 +75,10 @@ public Task<TResponse> DoRequestAsync<TResponse>(HttpMethod method, string path,
 		protected internal TRequestParams RequestParams<TRequestParams>(TRequestParams requestParams, string contentType = null, string accept = null)
 			where TRequestParams : class, IRequestParameters, new()
 		{
-			if (contentType.IsNullOrEmpty()) return requestParams;
+			if (contentType.IsNullOrEmpty() && accept.IsNullOrEmpty()) return requestParams;
 
 			requestParams ??= new TRequestParams();
-			if (requestParams.RequestConfiguration == null) requestParams.RequestConfiguration = new RequestConfiguration();
+			requestParams.RequestConfiguration ??= new RequestConfiguration();
 			if (!contentType.IsNullOrEmpty() && requestParams.RequestConfiguration.ContentType.IsNullOrEmpty())
 				requestParams.RequestConfiguration.ContentType = contentType;
 			if (!accept.IsNullOrEmpty() && requestParams.RequestConfiguration.Accept.IsNullOrEmpty())

src/Elasticsearch.Net/NamespacedClientProxy.cs

@@ -20,10 +20,14 @@ protected Task<TResponse> DoRequestAsync<TResponse>(HttpMethod post, string url,
 
 		protected string Url(FormattableString formattable) => _client.Url(formattable);
 
-		protected TRequestParams RequestParams<TRequestParams>(TRequestParams requestParams, string contentType = null)
+		protected TRequestParams RequestParams<TRequestParams>(TRequestParams requestParams, string contentType)
 			where TRequestParams : class, IRequestParameters, new()
 			=> _client.RequestParams(requestParams, contentType ?? ContentType, contentType ?? ContentType);
 
+		protected TRequestParams RequestParams<TRequestParams>(TRequestParams requestParams)
+			where TRequestParams : class, IRequestParameters, new()
+			=> _client.RequestParams(requestParams, ContentType, ContentType);
+
 		// ReSharper disable once UnassignedGetOnlyAutoProperty intended to be overridden
 		protected virtual string ContentType { get; }
 	}