fix:fix routes of gateway doesn't refresh bug. (#158)

* fix:fix routes of gateway doesn't refresh bug.

* docs:update CHANGELOG.md.

CHANGELOG.md

@@ -1,8 +1,4 @@
 # Change Log
 ---
 
-- [Feature: Support custom rate limit reject response info](https://github.com/Tencent/spring-cloud-tencent/pull/128)
-- [Feature: Optimize config server address](https://github.com/Tencent/spring-cloud-tencent/pull/130)
-- [Feature: Remove spring-javaformat-maven-plugin](https://github.com/Tencent/spring-cloud-tencent/pull/131)
-- [feat:refactor loadbalancer module as a basic module for router and circuit breaker.](https://github.com/Tencent/spring-cloud-tencent/pull/136)
-- [feat:enable distribute rate limit](https://github.com/Tencent/spring-cloud-tencent/pull/139)
+- [fix:fix routes of gateway doesn't refresh bug.](https://github.com/Tencent/spring-cloud-tencent/pull/158)

changes/changes-1.4.0.md

@@ -0,0 +1,8 @@
+# Change Log
+---
+
+- [Feature: Support custom rate limit reject response info](https://github.com/Tencent/spring-cloud-tencent/pull/128)
+- [Feature: Optimize config server address](https://github.com/Tencent/spring-cloud-tencent/pull/130)
+- [Feature: Remove spring-javaformat-maven-plugin](https://github.com/Tencent/spring-cloud-tencent/pull/131)
+- [feat:refactor loadbalancer module as a basic module for router and circuit breaker.](https://github.com/Tencent/spring-cloud-tencent/pull/136)
+- [feat:enable distribute rate limit](https://github.com/Tencent/spring-cloud-tencent/pull/139)

pom.xml

@@ -86,7 +86,7 @@
 
 	<properties>
 		<!-- Project revision -->
-		<revision>1.4.1-Hoxton.SR9</revision>
+		<revision>1.4.2-Hoxton.SR9-SNAPSHOT</revision>
 
 		<!-- Spring Cloud -->
 		<spring.cloud.version>Hoxton.SR9</spring.cloud.version>

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/DiscoveryConfigModifier.java

@@ -24,6 +24,8 @@ import com.tencent.polaris.api.config.consumer.ServiceRouterConfig;
 import com.tencent.polaris.factory.config.ConfigurationImpl;
 import com.tencent.polaris.plugins.router.healthy.RecoverRouterConfig;
 
+import org.springframework.beans.factory.annotation.Autowired;
+
 /**
  *  Spring Cloud Tencent config Override polaris config.
  *
@@ -31,6 +33,9 @@ import com.tencent.polaris.plugins.router.healthy.RecoverRouterConfig;
  */
 public class DiscoveryConfigModifier implements PolarisConfigModifier {
 
+	@Autowired
+	private PolarisDiscoveryProperties polarisDiscoveryProperties;
+
 	@Override
 	public void modify(ConfigurationImpl configuration) {
 		// Set excludeCircuitBreakInstances to false
@@ -41,6 +46,10 @@ public class DiscoveryConfigModifier implements PolarisConfigModifier {
 		// Update modified config to source properties
 		configuration.getConsumer().getServiceRouter()
 				.setPluginConfig(ServiceRouterConfig.DEFAULT_ROUTER_RECOVER, recoverRouterConfig);
+
+		// Set ServiceRefreshInterval
+		configuration.getConsumer().getLocalCache()
+				.setServiceListRefreshInterval(polarisDiscoveryProperties.getServiceListRefreshInterval());
 	}
 
 	@Override

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/PolarisDiscoveryProperties.java

@@ -105,6 +105,11 @@ public class PolarisDiscoveryProperties {
 	@Value("${spring.cloud.polaris.discovery.health-check-url:}")
 	private String healthCheckUrl;
 
+	/**
+	 * Millis interval of refresh of service info list. Default: 60000.
+	 */
+	private Long serviceListRefreshInterval = 60000L;
+
 	@Autowired
 	private Environment environment;
 
@@ -218,6 +223,14 @@ public class PolarisDiscoveryProperties {
 		this.healthCheckUrl = healthCheckUrl;
 	}
 
+	public Long getServiceListRefreshInterval() {
+		return serviceListRefreshInterval;
+	}
+
+	public void setServiceListRefreshInterval(Long serviceListRefreshInterval) {
+		this.serviceListRefreshInterval = serviceListRefreshInterval;
+	}
+
 	@Override
 	public String toString() {
 		return "PolarisProperties{" + "token='" + token + '\'' + ", namespace='"

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisDiscoveryHandler.java

@@ -33,6 +33,7 @@ import com.tencent.polaris.api.rpc.GetInstancesRequest;
 import com.tencent.polaris.api.rpc.GetServicesRequest;
 import com.tencent.polaris.api.rpc.InstancesResponse;
 import com.tencent.polaris.api.rpc.ServicesResponse;
+import com.tencent.polaris.client.api.SDKContext;
 import org.apache.commons.lang.StringUtils;
 
 import org.springframework.beans.factory.annotation.Autowired;
@@ -52,6 +53,9 @@ public class PolarisDiscoveryHandler {
 	@Autowired
 	private ProviderAPI providerAPI;
 
+	@Autowired
+	private SDKContext sdkContext;
+
 	@Autowired
 	private ConsumerAPI polarisConsumer;
 
@@ -114,6 +118,10 @@ public class PolarisDiscoveryHandler {
 		return providerAPI;
 	}
 
+	public SDKContext getSdkContext() {
+		return sdkContext;
+	}
+
 	/**
 	 * Return all service for given namespace.
 	 * @return service list

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/registry/PolarisServiceChangeListener.java

@@ -0,0 +1,62 @@
+package com.tencent.cloud.polaris.registry;
+
+import java.util.Set;
+import java.util.concurrent.atomic.AtomicInteger;
+import java.util.stream.Collectors;
+
+import com.google.common.collect.Sets;
+import com.tencent.polaris.api.plugin.registry.AbstractResourceEventListener;
+import com.tencent.polaris.api.pojo.RegistryCacheValue;
+import com.tencent.polaris.api.pojo.ServiceEventKey;
+import com.tencent.polaris.client.pojo.ServicesByProto;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import org.springframework.cloud.client.discovery.event.HeartbeatEvent;
+import org.springframework.context.ApplicationEventPublisher;
+import org.springframework.context.ApplicationEventPublisherAware;
+
+/**
+ * Change listener of Polaris service info.
+ *
+ * @author Haotian Zhang
+ */
+public class PolarisServiceChangeListener extends AbstractResourceEventListener implements ApplicationEventPublisherAware {
+
+	private static final Logger LOG = LoggerFactory.getLogger(PolarisServiceChangeListener.class);
+
+	private static final AtomicInteger INDEX = new AtomicInteger(0);
+
+	private ApplicationEventPublisher publisher;
+
+	@Override
+	public void onResourceUpdated(ServiceEventKey svcEventKey, RegistryCacheValue oldValue,
+			RegistryCacheValue newValue) {
+		if (newValue.getEventType() != ServiceEventKey.EventType.SERVICE) {
+			return;
+		}
+		if (oldValue instanceof ServicesByProto && newValue instanceof ServicesByProto) {
+			LOG.debug("receive service={} change event", svcEventKey);
+			Set<String> oldServiceInfoSet = ((ServicesByProto) oldValue).getServices().stream()
+					.map(i -> i.getNamespace() + "::" + i.getService()).collect(Collectors.toSet());
+			Set<String> newServiceInfoSet = ((ServicesByProto) newValue).getServices().stream()
+					.map(i -> i.getNamespace() + "::" + i.getService()).collect(Collectors.toSet());
+
+			Sets.SetView<String> addServiceInfoSetView = Sets.difference(newServiceInfoSet, oldServiceInfoSet);
+			Sets.SetView<String> deleteServiceInfoSetView = Sets.difference(oldServiceInfoSet, newServiceInfoSet);
+
+			if (addServiceInfoSetView.isEmpty() && deleteServiceInfoSetView.isEmpty()) {
+				return;
+			}
+			LOG.info("Service info is update. Add service of {}. Delete service of {}", addServiceInfoSetView, deleteServiceInfoSetView);
+
+			// Trigger reload of gateway route cache.
+			this.publisher.publishEvent(new HeartbeatEvent(this, INDEX.getAndIncrement()));
+		}
+	}
+
+	@Override
+	public void setApplicationEventPublisher(ApplicationEventPublisher applicationEventPublisher) {
+		this.publisher = applicationEventPublisher;
+	}
+}

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/registry/PolarisServiceRegistry.java

@@ -65,9 +65,11 @@ public class PolarisServiceRegistry implements ServiceRegistry<Registration> {
 
 	private final ScheduledExecutorService heartbeatExecutor;
 
+	private final PolarisServiceChangeListener polarisServiceChangeListener;
+
 	public PolarisServiceRegistry(PolarisDiscoveryProperties polarisDiscoveryProperties,
 			PolarisDiscoveryHandler polarisDiscoveryHandler,
-			MetadataLocalProperties metadataLocalProperties) {
+			MetadataLocalProperties metadataLocalProperties, PolarisServiceChangeListener polarisServiceChangeListener) {
 		this.polarisDiscoveryProperties = polarisDiscoveryProperties;
 		this.polarisDiscoveryHandler = polarisDiscoveryHandler;
 		this.metadataLocalProperties = metadataLocalProperties;
@@ -79,6 +81,8 @@ public class PolarisServiceRegistry implements ServiceRegistry<Registration> {
 		else {
 			this.heartbeatExecutor = null;
 		}
+
+		this.polarisServiceChangeListener = polarisServiceChangeListener;
 	}
 
 	@Override
@@ -116,6 +120,10 @@ public class PolarisServiceRegistry implements ServiceRegistry<Registration> {
 				// Start the heartbeat thread after the registration is successful.
 				heartbeat(heartbeatRequest);
 			}
+
+			// Register service change listener
+			polarisDiscoveryHandler.getSdkContext().getExtensions().getLocalRegistry()
+					.registerResourceListener(polarisServiceChangeListener);
 		}
 		catch (Exception e) {
 			log.error("polaris registry, {} register failed...{},",

spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/registry/PolarisServiceRegistryAutoConfiguration.java

@@ -27,6 +27,7 @@ import com.tencent.polaris.client.api.SDKContext;
 
 import org.springframework.boot.autoconfigure.AutoConfigureAfter;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
+import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
 import org.springframework.boot.context.properties.EnableConfigurationProperties;
 import org.springframework.cloud.client.serviceregistry.AutoServiceRegistrationAutoConfiguration;
@@ -52,11 +53,10 @@ public class PolarisServiceRegistryAutoConfiguration {
 
 	@Bean
 	public PolarisServiceRegistry polarisServiceRegistry(
-			PolarisDiscoveryProperties polarisDiscoveryProperties,
-			PolarisDiscoveryHandler polarisDiscoveryHandler,
-			MetadataLocalProperties metadataLocalProperties) {
+			PolarisDiscoveryProperties polarisDiscoveryProperties, PolarisDiscoveryHandler polarisDiscoveryHandler,
+			MetadataLocalProperties metadataLocalProperties, PolarisServiceChangeListener polarisServiceChangeListener) {
 		return new PolarisServiceRegistry(polarisDiscoveryProperties,
-				polarisDiscoveryHandler, metadataLocalProperties);
+				polarisDiscoveryHandler, metadataLocalProperties, polarisServiceChangeListener);
 	}
 
 	@Bean
@@ -78,4 +78,9 @@ public class PolarisServiceRegistryAutoConfiguration {
 				autoServiceRegistrationProperties, registration);
 	}
 
+	@Bean
+	@ConditionalOnMissingBean
+	public PolarisServiceChangeListener polarisServiceChangeListener() {
+		return new PolarisServiceChangeListener();
+	}
 }

spring-cloud-starter-tencent-polaris-discovery/src/main/resources/META-INF/additional-spring-configuration-metadata.json

@@ -59,6 +59,12 @@
       "type": "java.lang.Integer",
       "defaultValue": 100,
       "description": "the weight of polaris instance , use to load-balance."
+    },
+    {
+      "name": "spring.cloud.polaris.discovery.service-list-refresh-interval",
+      "type": "java.lang.Long",
+      "defaultValue": 60000,
+      "description": "Millis interval of refresh of service info list. Default: 60000."
     }
   ]
 }

spring-cloud-tencent-dependencies/pom.xml

@@ -70,7 +70,7 @@
 	</developers>
 
 	<properties>
-		<revision>1.4.1-Hoxton.SR9</revision>
+		<revision>1.4.2-Hoxton.SR9-SNAPSHOT</revision>
 		<polaris.version>1.5.1</polaris.version>
 		<powermock.version>2.0.0</powermock.version>
 

spring-cloud-tencent-examples/polaris-gateway-example/gateway-scg-service/src/main/resources/bootstrap.yml

@@ -15,19 +15,52 @@ spring:
       address: grpc://127.0.0.1:8091
       namespace: default
       enabled: true
+      discovery:
+        service-list-refresh-interval: 1000
     gateway:
       discovery:
         locator:
           enabled: true
-          lowerCaseServiceId: false
-      routes:
-        - id: GatewayCalleeService
-          uri: lb://GatewayCalleeService
-          predicates:
-            - Path=/GatewayCalleeService/**
-          filters:
-            - StripPrefix=1
+          'predicates[0]':
+            name: Path
+            args:
+              patterns: '''/'' + serviceId + ''/**'''
+          'filters[0]':
+            name: RewritePath
+            args:
+              regexp: '''/'' + serviceId + ''/(?<remaining>.*)'''
+              replacement: '''/$\{remaining}'''
+          'filters[1]':
+            name: Retry
+            args:
+              retries: 3
+              exceptions:
+                '[0]': '''java.net.ConnectException'''
+                '[1]': '''java.io.IOException'''
+              statuses:
+                '[0]': '''BAD_GATEWAY'''
+                '[1]': '''SERVICE_UNAVAILABLE'''
+              series:
+                '[0]': '''CLIENT_ERROR'''
+              methods:
+                '[0]': '''GET'''
+                '[1]': '''POST'''
+                '[2]': '''PUT'''
+                '[3]': '''DELETE'''
+              backoff:
+                firstBackoff: '''100ms'''
+                maxBackoff: '''500ms'''
+                factor: 2
+                basedOnPreviousValue: false
+#      routes:
+#        - id: GatewayCalleeService
+#          uri: lb://GatewayCalleeService
+#          predicates:
+#            - Path=/GatewayCalleeService/**
+#          filters:
+#            - StripPrefix=1
 
 logging:
   level:
     org.springframework.cloud.gateway: info
+    com.tencent.cloud.polaris: debug

src/checkstyle/checkstyle-suppressions.xml

@@ -5,4 +5,5 @@
 <suppressions>
 	<suppress files=".*" checks="RegexpHeader"/>
 	<suppress files=".*" checks="HideUtilityClassConstructor"/>
+	<suppress files=".*" checks="IllegalImport"/>
 </suppressions>