fix:fix feign report -1 port bug.

Tencent · Aug 8, 2023 · 264b654 · 264b654
1 parent 51504c2
commit 264b654
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/pom.xml b/pom.xml
@@ -89,7 +89,7 @@
 
 	<properties>
 		<!-- Project revision -->
-		<revision>1.11.8-Hoxton.SR12</revision>
+		<revision>1.11.9-Hoxton.SR12</revision>
 
 		<!-- Spring Framework -->
 		<spring.framework.version>5.2.22.RELEASE</spring.framework.version>

diff --git a/...c/test/java/com/tencent/cloud/metadata/core/EncodeTransferMedataFeignInterceptorTest.java b/...c/test/java/com/tencent/cloud/metadata/core/EncodeTransferMedataFeignInterceptorTest.java
@@ -47,7 +47,7 @@
 @ExtendWith(SpringExtension.class)
 @SpringBootTest(webEnvironment = DEFINED_PORT,
 		classes = EncodeTransferMedataFeignInterceptorTest.TestApplication.class,
-		properties = {"server.port=8081",
+		properties = {"server.port=18081",
 				"spring.config.location = classpath:application-test.yml",
 				"spring.main.web-application-type = servlet",
 				"spring.cloud.gateway.enabled = false"})
@@ -77,7 +77,7 @@ public String test() {
 			return MetadataContextHolder.get().getContext(MetadataContext.FRAGMENT_TRANSITIVE, "b");
 		}
 
-		@FeignClient(name = "test-feign", url = "http://localhost:8081")
+		@FeignClient(name = "test-feign", url = "http://localhost:18081")
 		public interface TestFeign {
 
 			@RequestMapping("/test")

diff --git a/spring-cloud-tencent-dependencies/pom.xml b/spring-cloud-tencent-dependencies/pom.xml
@@ -70,7 +70,7 @@
 	</developers>
 
 	<properties>
-		<revision>1.11.8-Hoxton.SR12</revision>
+		<revision>1.11.9-Hoxton.SR12</revision>
 
 		<!-- Dependencies -->
 		<polaris.version>1.12.11</polaris.version>

diff --git a/...nhancement/src/main/java/com/tencent/cloud/rpc/enhancement/feign/EnhancedFeignClient.java b/...nhancement/src/main/java/com/tencent/cloud/rpc/enhancement/feign/EnhancedFeignClient.java
@@ -89,6 +89,8 @@ public Response execute(Request request, Options options) throws IOException {
 
 			DefaultServiceInstance serviceInstance = new DefaultServiceInstance(request.requestTemplate().feignTarget()
 					.name(), url.getHost(), url.getPort(), url.getScheme().equals("https"));
+			// -1 means access directly by url, and use http default port number 80
+			serviceInstance.setPort(serviceInstance.getPort() == -1 ? 80 : serviceInstance.getPort());
 			enhancedPluginContext.setServiceInstance(serviceInstance);
 
 			// Run post enhanced plugins.