fix: fix npe when add circuitbreak module without feign.hystrix.enable=true (#1292)

CHANGELOG.md

@@ -9,4 +9,5 @@
 - [feat:add polaris ThreadLocal plugin.](https://github.com/Tencent/spring-cloud-tencent/pull/1255)
 - [feat:upgrade jackson version.](https://github.com/Tencent/spring-cloud-tencent/pull/1257)
 - [fix:fix wrong report when using Zuul with instance not found exception.](https://github.com/Tencent/spring-cloud-tencent/pull/1283)
-- [feat: merge lane router and lossless features from 2023](https://github.com/Tencent/spring-cloud-tencent/pull/1288)
+- [feat: merge lane router and lossless features from 2023](https://github.com/Tencent/spring-cloud-tencent/pull/1288)
+- [fix: fix npe when add circuitbreak module without feign.hystrix.enable=true](https://github.com/Tencent/spring-cloud-tencent/pull/1292)

spring-cloud-starter-tencent-polaris-circuitbreaker/src/main/java/com/tencent/cloud/polaris/circuitbreaker/feign/PolarisFeignCircuitBreakerInvocationHandler.java

@@ -113,36 +113,41 @@ public class PolarisFeignCircuitBreakerInvocationHandler implements InvocationHa
 		else if ("toString".equals(method.getName())) {
 			return toString();
 		}
-
-		String circuitName = circuitBreakerNameResolver.resolveCircuitBreakerName(feignClientName, target, method);
-		CircuitBreaker circuitBreaker = factory.create(circuitName);
 		Supplier<Object> supplier = asSupplier(method, args);
-		Function<Throwable, Object> fallbackFunction;
-		if (this.nullableFallbackFactory != null) {
-			fallbackFunction = throwable -> {
-				Object fallback = this.nullableFallbackFactory.create(throwable);
-				try {
-					return this.fallbackMethodMap.get(method).invoke(fallback, args);
-				}
-				catch (Exception exception) {
-					unwrapAndRethrow(exception);
-				}
-				return null;
-			};
+		if (circuitBreakerNameResolver != null) {
+			String circuitName = circuitBreakerNameResolver.resolveCircuitBreakerName(feignClientName, target, method);
+			CircuitBreaker circuitBreaker = factory.create(circuitName);
+
+			Function<Throwable, Object> fallbackFunction;
+			if (this.nullableFallbackFactory != null) {
+				fallbackFunction = throwable -> {
+					Object fallback = this.nullableFallbackFactory.create(throwable);
+					try {
+						return this.fallbackMethodMap.get(method).invoke(fallback, args);
+					}
+					catch (Exception exception) {
+						unwrapAndRethrow(exception);
+					}
+					return null;
+				};
+			}
+			else {
+				fallbackFunction = throwable -> {
+					PolarisCircuitBreakerFallbackFactory.DefaultFallback fallback =
+							(PolarisCircuitBreakerFallbackFactory.DefaultFallback) new PolarisCircuitBreakerFallbackFactory(this.decoder).create(throwable);
+					return fallback.fallback(method);
+				};
+			}
+			try {
+				return circuitBreaker.run(supplier, fallbackFunction);
+			}
+			catch (FallbackWrapperException e) {
+				// unwrap And Rethrow
+				throw e.getCause();
+			}
 		}
 		else {
-			fallbackFunction = throwable -> {
-				PolarisCircuitBreakerFallbackFactory.DefaultFallback fallback =
-						(PolarisCircuitBreakerFallbackFactory.DefaultFallback) new PolarisCircuitBreakerFallbackFactory(this.decoder).create(throwable);
-				return fallback.fallback(method);
-			};
-		}
-		try {
-			return circuitBreaker.run(supplier, fallbackFunction);
-		}
-		catch (FallbackWrapperException e) {
-			// unwrap And Rethrow
-			throw e.getCause();
+			return supplier.get();
 		}
 	}