[SCB-2881] support setting hibernate validation configuration (#4343)

Author: chengyouling

core/src/main/java/org/apache/servicecomb/core/filter/impl/ParameterValidatorFilter.java

@@ -17,9 +17,11 @@
 package org.apache.servicecomb.core.filter.impl;
 
 import java.lang.reflect.Method;
+import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 
+import org.apache.servicecomb.config.ConfigUtil;
 import org.apache.servicecomb.core.Invocation;
 import org.apache.servicecomb.core.filter.AbstractFilter;
 import org.apache.servicecomb.core.filter.Filter;
@@ -36,6 +38,7 @@
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.InitializingBean;
 
+import jakarta.validation.Configuration;
 import jakarta.validation.ConstraintViolation;
 import jakarta.validation.ConstraintViolationException;
 import jakarta.validation.Valid;
@@ -71,6 +74,8 @@ private static class Model {
 
   public static final String NAME = "validator";
 
+  public static final String HIBERNATE_VALIDATE_PREFIX = "hibernate.validator";
+
   public static final String ENABLE_EL = "servicecomb.filters.validation.useResourceBundleMessageInterpolator";
 
   protected ExecutableValidator validator;
@@ -107,11 +112,17 @@ private void initValidate() {
   }
 
   protected ValidatorFactory createValidatorFactory() {
-    return Validation.byProvider(HibernateValidator.class)
+    Configuration<?> validatorConfiguration = Validation.byProvider(HibernateValidator.class)
         .configure()
         .propertyNodeNameProvider(new JacksonPropertyNodeNameProvider())
-        .messageInterpolator(messageInterpolator())
-        .buildValidatorFactory();
+        .messageInterpolator(messageInterpolator());
+    Map<String, String> properties = ConfigUtil.stringPropertiesWithPrefix(environment, HIBERNATE_VALIDATE_PREFIX);
+    if (!properties.isEmpty()) {
+      for (Map.Entry<String, String> entry : properties.entrySet()) {
+        validatorConfiguration.addProperty(entry.getKey(), entry.getValue());
+      }
+    }
+    return validatorConfiguration.buildValidatorFactory();
   }
 
   protected AbstractMessageInterpolator messageInterpolator() {

core/src/test/java/org/apache/servicecomb/core/filter/impl/ParameterValidatorFilterTest.java

@@ -42,7 +42,8 @@
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
-import org.springframework.core.env.Environment;
+import org.springframework.core.env.ConfigurableEnvironment;
+import org.springframework.core.env.MutablePropertySources;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
 
@@ -96,14 +97,16 @@ public void op(@NotNull(message = "not null") String query, @Valid Model model)
   @Mocked
   SwaggerProducerOperation operation;
 
-  Environment environment;
+  ConfigurableEnvironment environment;
 
   @Before
   public void setUp() throws Exception {
-    environment = Mockito.mock(Environment.class);
+    environment = Mockito.mock(ConfigurableEnvironment.class);
     SCBEngine engine = SCBBootstrap.createSCBEngineForTest(environment);
+    MutablePropertySources sources = new MutablePropertySources();
 
     Mockito.when(environment.getProperty(ENABLE_EL, boolean.class, false)).thenReturn(false);
+    Mockito.when(environment.getPropertySources()).thenReturn(sources);
     filter.setEnvironment(environment);
     filter.afterPropertiesSet();
     engine.setEnvironment(environment);

swagger/swagger-invocation/invocation-core/src/main/java/org/apache/servicecomb/swagger/engine/SwaggerProducerOperation.java

@@ -18,13 +18,10 @@
 
 import java.lang.reflect.Method;
 import java.lang.reflect.Type;
-import java.util.List;
 import java.util.Map;
 
-import org.apache.servicecomb.foundation.common.utils.SPIServiceUtils;
 import org.apache.servicecomb.swagger.generator.core.model.SwaggerOperation;
 import org.apache.servicecomb.swagger.invocation.arguments.producer.ProducerArgumentsMapper;
-import org.apache.servicecomb.swagger.invocation.extension.ProducerInvokeExtension;
 import org.apache.servicecomb.swagger.invocation.response.producer.ProducerResponseMapper;
 
 public class SwaggerProducerOperation {
@@ -47,9 +44,6 @@ public class SwaggerProducerOperation {
 
   private ProducerResponseMapper responseMapper;
 
-  private final List<ProducerInvokeExtension> producerInvokeExtenstionList =
-      SPIServiceUtils.getSortedService(ProducerInvokeExtension.class);
-
   public String getOperationId() {
     return swaggerOperation.getOperationId();
   }
@@ -111,10 +105,6 @@ public void setResponseMapper(ProducerResponseMapper responseMapper) {
     this.responseMapper = responseMapper;
   }
 
-  public List<ProducerInvokeExtension> getProducerInvokeExtenstionList() {
-    return this.producerInvokeExtenstionList;
-  }
-
   public Type getSwaggerParameterType(String name) {
     return this.swaggerParameterTypes.get(name);
   }