fix: extensionsParamChecker

alibaba · Nov 30, 2024 · 589afad · 589afad
1 parent 2678342
commit 589afad
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/common/src/main/java/com/alibaba/nacos/common/paramcheck/DefaultParamChecker.java b/common/src/main/java/com/alibaba/nacos/common/paramcheck/DefaultParamChecker.java
@@ -72,12 +72,17 @@ public ParamCheckResponse checkParamInfoList(List<ParamInfo> paramInfos, Functio
         }
         for (ParamInfo paramInfo : paramInfos) {
             paramCheckResponse = checkParamInfoFormat(paramInfo);
-            if (Objects.nonNull(extensionsParamChecker)) {
-                paramCheckResponse = extensionsParamChecker.apply(paramInfo);
-            }
             if (!paramCheckResponse.isSuccess()) {
                 return paramCheckResponse;
             }
+            // if extensionsParamChecker exists, check params
+            if (Objects.nonNull(extensionsParamChecker)) {
+                paramCheckResponse = extensionsParamChecker.apply(paramInfo);
+                // if not success ,return
+                if (!paramCheckResponse.isSuccess()) {
+                    return paramCheckResponse;
+                }
+            }
         }
         paramCheckResponse.setSuccess(true);
         return paramCheckResponse;