fix: fix RouterLabelRestTemplateInterceptor add response headers exce…

…ption with httpclient5 (#1239)
Tencent · Feb 23, 2024 · 51fb3a1 · 51fb3a1
1 parent 44426a8
commit 51fb3a1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,4 @@
 # Change Log
 ---
 
+[fix: fix RouterLabelRestTemplateInterceptor add response headers exception with httpclient5.](https://github.com/Tencent/spring-cloud-tencent/pull/1239)
diff --git a/...ava/com/tencent/cloud/polaris/router/resttemplate/RouterLabelRestTemplateInterceptor.java b/...ava/com/tencent/cloud/polaris/router/resttemplate/RouterLabelRestTemplateInterceptor.java
@@ -45,6 +45,7 @@
 import org.slf4j.LoggerFactory;
 
 import org.springframework.core.Ordered;
+import org.springframework.http.HttpHeaders;
 import org.springframework.http.HttpRequest;
 import org.springframework.http.client.ClientHttpRequestExecution;
 import org.springframework.http.client.ClientHttpRequestInterceptor;
@@ -104,7 +105,8 @@ public ClientHttpResponse intercept(@NonNull HttpRequest request, @NonNull byte[
 		ClientHttpResponse response = clientHttpRequestExecution.execute(request, body);
 
 		if (!CollectionUtils.isEmpty(request.getHeaders().get(RouterConstant.ROUTER_LABEL_HEADER))) {
-			response.getHeaders().addAll(RouterConstant.ROUTER_LABEL_HEADER, Objects.requireNonNull(request.getHeaders()
+			HttpHeaders responseHeaders = HttpHeaders.writableHttpHeaders(response.getHeaders());
+			responseHeaders.addAll(RouterConstant.ROUTER_LABEL_HEADER, Objects.requireNonNull(request.getHeaders()
 					.get(RouterConstant.ROUTER_LABEL_HEADER)));
 		}