fix:fix RestTemplate trace attribute loss bug.

Author: SkyeBeFreeman

CHANGELOG.md

@@ -15,3 +15,4 @@
 - fix:fix PolarisContextProperties instantiated twice causing NPE.
 - fix:fix ipv6 instance not found bug.
 - fix:fix tsf lossless not working bug.
+- fix:fix RestTemplate trace attribute loss bug.

spring-cloud-starter-tencent-polaris-circuitbreaker/src/main/java/com/tencent/cloud/polaris/circuitbreaker/reporter/CircuitBreakerPlugin.java

@@ -70,8 +70,8 @@ public void run(EnhancedPluginContext context) throws Throwable {
 		EnhancedRequestContext request = context.getRequest();
 		EnhancedResponseContext response = context.getResponse();
 
-		String host = request.getServiceUrl() != null ? request.getServiceUrl().getHost() : request.getUrl().getHost();
-		String path = request.getServiceUrl() != null ? request.getServiceUrl().getPath() : request.getUrl().getPath();
+		String host = request.getHost();
+		String path = request.getPath();
 		String governanceNamespace = StringUtils.isNotEmpty(request.getGovernanceNamespace()) ? request.getGovernanceNamespace() : MetadataContext.LOCAL_NAMESPACE;
 
 		String httpMethod = request.getHttpMethod().name();

spring-cloud-tencent-plugin-starters/spring-cloud-starter-tencent-trace-plugin/src/main/java/com/tencent/cloud/plugin/trace/attribute/PolarisSpanAttributesProvider.java

@@ -84,7 +84,7 @@ public Map<String, String> getClientBaggageAttributes(EnhancedPluginContext cont
 		}
 		attributes.put("http.port", CalleeMetadataContainerGroup.getStaticApplicationMetadataContainer()
 				.getRawMetadataStringValue(MetadataConstants.LOCAL_PORT));
-		attributes.put("net.peer.service", context.getTargetServiceInstance().getServiceId());
+		attributes.put("net.peer.service", context.getRequest().getHost());
 
 		String serviceLane = metadataContext.getMetadataContainer(MetadataType.MESSAGE, false)
 				.getRawMetadataMapValue(MessageMetadataContainer.LABEL_MAP_KEY_HEADER, TRAFFIC_STAIN_LABEL);

spring-cloud-tencent-plugin-starters/spring-cloud-starter-tencent-trace-plugin/src/main/java/com/tencent/cloud/plugin/trace/attribute/tsf/TsfSpanAttributesProvider.java

@@ -33,8 +33,8 @@ public class TsfSpanAttributesProvider implements SpanAttributesProvider {
 	@Override
 	public Map<String, String> getClientBaggageAttributes(EnhancedPluginContext context) {
 		Map<String, String> attributes = new HashMap<>();
-		if (null != context.getRequest().getUrl()) {
-			attributes.put("remoteInterface", context.getRequest().getUrl().getPath());
+		if (context.getRequest() != null && StringUtils.isNotBlank(context.getRequest().getPath())) {
+			attributes.put("remoteInterface", context.getRequest().getPath());
 		}
 		ServiceInstance targetServiceInstance = context.getTargetServiceInstance();
 		if (null != targetServiceInstance && CollectionUtils.isNotEmpty(targetServiceInstance.getMetadata())) {

spring-cloud-tencent-rpc-enhancement/src/main/java/com/tencent/cloud/rpc/enhancement/plugin/EnhancedRequestContext.java

@@ -39,6 +39,10 @@ public class EnhancedRequestContext {
 
 	private String governanceNamespace;
 
+	public static EnhancedContextRequestBuilder builder() {
+		return new EnhancedContextRequestBuilder();
+	}
+
 	public HttpMethod getHttpMethod() {
 		return httpMethod;
 	}
@@ -79,8 +83,24 @@ public void setGovernanceNamespace(String governanceNamespace) {
 		this.governanceNamespace = governanceNamespace;
 	}
 
-	public static EnhancedContextRequestBuilder builder() {
-		return new EnhancedContextRequestBuilder();
+	public String getHost() {
+		if (serviceUrl != null) {
+			return serviceUrl.getHost();
+		}
+		if (url != null) {
+			return url.getHost();
+		}
+		return null;
+	}
+
+	public String getPath() {
+		if (serviceUrl != null) {
+			return serviceUrl.getPath();
+		}
+		if (url != null) {
+			return url.getPath();
+		}
+		return null;
 	}
 
 	@Override