Do not send null HTTP header value in JdkClientHttpRequest

Prior to this commit, the `JdkClientHttpRequest` would add all values
from `HttpHeaders` to the native request builder. This could cause
`NullPointerException` being thrown at runtime because the `HttpClient`
does not support that.

This commit replicates a fix that was applied to the
`SimpleClientHttpRequest`, turning null values into empty "".

Fixes gh-35996

Author: bclozel

spring-web/src/main/java/org/springframework/http/client/JdkClientHttpRequest.java

@@ -155,7 +155,7 @@ private HttpRequest buildRequest(HttpHeaders headers, @Nullable Body body) {
 		headers.forEach((headerName, headerValues) -> {
 			if (!DISALLOWED_HEADERS.contains(headerName.toLowerCase(Locale.ROOT))) {
 				for (String headerValue : headerValues) {
-					builder.header(headerName, headerValue);
+					builder.header(headerName, (headerValue != null) ? headerValue : "");
 				}
 			}
 		});

spring-web/src/main/java/org/springframework/http/client/OkHttp3ClientHttpRequest.java

@@ -90,7 +90,7 @@ else if (okhttp3.internal.http.HttpMethod.requiresRequestBody(getMethod().name()
 		builder.method(this.method.name(), requestBody);
 		headers.forEach((headerName, headerValues) -> {
 			for (String headerValue : headerValues) {
-				builder.addHeader(headerName, headerValue);
+				builder.addHeader(headerName, (headerValue != null) ? headerValue : "");
 			}
 		});
 		Request request = builder.build();

spring-web/src/test/java/org/springframework/web/client/RestClientIntegrationTests.java

@@ -961,6 +961,29 @@ void defaultHeaders(ClientHttpRequestFactory requestFactory) {
 		expectRequest(request -> assertThat(request.getHeader("foo")).isEqualTo("bar"));
 	}
 
+
+	@ParameterizedRestClientTest
+	void sendNullHeaderValue(ClientHttpRequestFactory requestFactory) throws IOException {
+		startServer(requestFactory);
+
+		prepareResponse(builder -> builder
+				.setHeader("Content-Type", "text/plain").setBody("Hello Spring!"));
+
+		String result = this.restClient.get()
+				.uri("/greeting")
+				.httpRequest(request -> request.getHeaders().add("X-Test-Header", null))
+				.retrieve()
+				.body(String.class);
+
+		assertThat(result).isEqualTo("Hello Spring!");
+
+		expectRequestCount(1);
+		expectRequest(request -> {
+			assertThat(request.getHeaders().get("X-Test-Header")).isNullOrEmpty();
+			assertThat(request.getPath()).isEqualTo("/greeting");
+		});
+	}
+
 	@ParameterizedRestClientTest
 	void defaultRequest(ClientHttpRequestFactory requestFactory) {
 		startServer(requestFactory);