🔧 避免各种依赖的地方组件造成 BeanPostProcessorChecker 警告

pig-mesh · Apr 19, 2024 · e9e0a03 · e9e0a03
1 parent 09cbd8d
commit e9e0a03
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 33 deletions.
diff --git a/...mon-feign/src/main/java/org/springframework/cloud/openfeign/PigFeignClientsRegistrar.java b/...mon-feign/src/main/java/org/springframework/cloud/openfeign/PigFeignClientsRegistrar.java
@@ -34,7 +34,6 @@
 import org.springframework.core.io.support.SpringFactoriesLoader;
 import org.springframework.core.type.AnnotationMetadata;
 import org.springframework.lang.Nullable;
-import org.springframework.util.ClassUtils;
 import org.springframework.util.StringUtils;
 
 import java.util.ArrayList;
@@ -68,11 +67,12 @@ public void setBeanClassLoader(ClassLoader classLoader) {
 	}
 
 	private void registerFeignClients(BeanDefinitionRegistry registry) {
-		List<String> feignClients = new ArrayList<>();
+
+		List<String> feignClients = new ArrayList<>(
+				SpringFactoriesLoader.loadFactoryNames(getSpringFactoriesLoaderFactoryClass(), getBeanClassLoader()));
 
 		// 支持 springboot 2.7 + 最新版本的配置方式
 		ImportCandidates.load(FeignClient.class, getBeanClassLoader()).forEach(feignClients::add);
-
 		// 如果 spring.factories 里为空
 		if (feignClients.isEmpty()) {
 			return;
@@ -93,13 +93,12 @@ private void registerFeignClients(BeanDefinitionRegistry registry) {
 					continue;
 				}
 
-				registerClientConfiguration(registry, getClientName(attributes), className,
-						attributes.get("configuration"));
+				registerClientConfiguration(registry, getClientName(attributes), attributes.get("configuration"));
 
 				validate(attributes);
 				BeanDefinitionBuilder definition = BeanDefinitionBuilder
 					.genericBeanDefinition(FeignClientFactoryBean.class);
-				definition.addPropertyValue("url", getUrl(attributes));
+				definition.addPropertyValue("url", getUrl(registry, attributes));
 				definition.addPropertyValue("path", getPath(attributes));
 				String name = getName(attributes);
 				definition.addPropertyValue("name", name);
@@ -116,13 +115,8 @@ private void registerFeignClients(BeanDefinitionRegistry registry) {
 				}
 
 				definition.addPropertyValue("type", className);
-				definition.addPropertyValue("dismiss404",
-						Boolean.parseBoolean(String.valueOf(attributes.get("dismiss404"))));
-				Object fallbackFactory = attributes.get("fallbackFactory");
-				if (fallbackFactory != null) {
-					definition.addPropertyValue("fallbackFactory", fallbackFactory instanceof Class ? fallbackFactory
-							: ClassUtils.resolveClassName(fallbackFactory.toString(), null));
-				}
+				definition.addPropertyValue("decode404", attributes.get("decode404"));
+				definition.addPropertyValue("fallback", attributes.get("fallback"));
 				definition.addPropertyValue("fallbackFactory", attributes.get("fallbackFactory"));
 				definition.setAutowireMode(AbstractBeanDefinition.AUTOWIRE_BY_TYPE);
 
@@ -197,7 +191,7 @@ private String resolve(String value) {
 		return value;
 	}
 
-	private String getUrl(Map<String, Object> attributes) {
+	private String getUrl(BeanDefinitionRegistry registry, Map<String, Object> attributes) {
 
 		// 如果是单体项目自动注入 & url 为空
 		Boolean isMicro = environment.getProperty("spring.cloud.nacos.discovery.enabled", Boolean.class, true);
@@ -206,15 +200,7 @@ private String getUrl(Map<String, Object> attributes) {
 			return null;
 		}
 
-		Object objUrl = attributes.get("url");
-
-		String url = "";
-		if (StringUtils.hasText(objUrl.toString())) {
-			url = resolve(objUrl.toString());
-		}
-		else {
-			url = resolve(BASE_URL);
-		}
+		String url = resolve(BASE_URL);
 
 		return FeignClientsRegistrar.getUrl(url);
 	}
@@ -267,16 +253,6 @@ private void registerClientConfiguration(BeanDefinitionRegistry registry, Object
 				builder.getBeanDefinition());
 	}
 
-	private void registerClientConfiguration(BeanDefinitionRegistry registry, Object name, Object className,
-			Object configuration) {
-		BeanDefinitionBuilder builder = BeanDefinitionBuilder.genericBeanDefinition(FeignClientSpecification.class);
-		builder.addConstructorArgValue(name);
-		builder.addConstructorArgValue(className);
-		builder.addConstructorArgValue(configuration);
-		registry.registerBeanDefinition(name + "." + FeignClientSpecification.class.getSimpleName(),
-				builder.getBeanDefinition());
-	}
-
 	@Override
 	public void setEnvironment(Environment environment) {
 		this.environment = environment;

diff --git a/...mon-log/src/main/java/com/pig4cloud/pig/common/log/init/ApplicationLoggerInitializer.java b/...mon-log/src/main/java/com/pig4cloud/pig/common/log/init/ApplicationLoggerInitializer.java
@@ -39,6 +39,10 @@ public void postProcessEnvironment(ConfigurableEnvironment environment, SpringAp
 
 		// 避免 sentinel 1.8.4+ 心跳日志过大
 		System.setProperty("csp.sentinel.log.level", "OFF");
+
+		// 避免各种依赖的地方组件造成 BeanPostProcessorChecker 警告
+		System.setProperty("logging.level.org.springframework.context.support.PostProcessorRegistrationDelegate",
+				"ERROR");
 	}
 
 	@Override