Duplicate ServiceBean Issue (#721)
* Polish apache/dubbo-spring-boot-project#718 : [Samples] Introducing the samples deployed the external Servlet container
* Polish apache/dubbo-spring-boot-project#718 : [Samples] Introducing the samples deployed the external Servlet container
* Polish apache/dubbo-spring-boot-project#718 : [Samples] Introducing the samples deployed the external Servlet container
* Polish apache/dubbo-spring-boot-project#685 : Dubbo2.7.5: Duplicate ServiceBean found
diff --git a/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboAutoConfiguration.java b/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboAutoConfiguration.java
index 6a190cb..6f130e8 100644
--- a/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboAutoConfiguration.java
+++ b/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/autoconfigure/DubboAutoConfiguration.java
@@ -23,6 +23,7 @@
import org.apache.dubbo.config.spring.beans.factory.annotation.ServiceClassPostProcessor;
import org.apache.dubbo.config.spring.context.annotation.DubboConfigConfiguration;
import org.apache.dubbo.config.spring.context.annotation.EnableDubboConfig;
+import org.apache.dubbo.spring.boot.beans.factory.config.ServiceBeanIdConflictProcessor;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.beans.factory.annotation.Qualifier;
@@ -61,6 +62,7 @@
@Configuration
@AutoConfigureAfter(DubboRelaxedBindingAutoConfiguration.class)
@EnableConfigurationProperties(DubboConfigurationProperties.class)
+@Import(ServiceBeanIdConflictProcessor.class)
public class DubboAutoConfiguration {
/**
diff --git a/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/beans/factory/config/ServiceBeanIdConflictProcessor.java b/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/beans/factory/config/ServiceBeanIdConflictProcessor.java
new file mode 100644
index 0000000..b6bf04b
--- /dev/null
+++ b/dubbo-spring-boot-compatible/autoconfigure/src/main/java/org/apache/dubbo/spring/boot/beans/factory/config/ServiceBeanIdConflictProcessor.java
@@ -0,0 +1,115 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.dubbo.spring.boot.beans.factory.config;
+
+import org.apache.dubbo.config.ServiceConfig;
+import org.apache.dubbo.config.spring.ServiceBean;
+
+import org.springframework.beans.BeansException;
+import org.springframework.beans.factory.DisposableBean;
+import org.springframework.beans.factory.support.MergedBeanDefinitionPostProcessor;
+import org.springframework.beans.factory.support.RootBeanDefinition;
+import org.springframework.context.annotation.CommonAnnotationBeanPostProcessor;
+import org.springframework.core.Ordered;
+import org.springframework.core.PriorityOrdered;
+
+import java.util.HashMap;
+import java.util.LinkedHashSet;
+import java.util.Map;
+import java.util.Objects;
+import java.util.Set;
+
+import static org.springframework.util.ClassUtils.getUserClass;
+import static org.springframework.util.ClassUtils.isAssignable;
+
+/**
+ * The post-processor for resolving the id conflict of {@link ServiceBean} when an interface is
+ * implemented by multiple services with different groups or versions that are exported on one provider
+ * <p>
+ * Current implementation is a temporary resolution, and will be removed in the future.
+ *
+ * @see CommonAnnotationBeanPostProcessor
+ * @since 2.7.7
+ * @deprecated
+ */
+public class ServiceBeanIdConflictProcessor implements MergedBeanDefinitionPostProcessor, DisposableBean, PriorityOrdered {
+
+ /**
+ * The key is the class names of interfaces that were exported by {@link ServiceBean}
+ * The value is bean names of {@link ServiceBean} or {@link ServiceConfig}.
+ */
+ private Map<String, String> interfaceNamesToBeanNames = new HashMap<>();
+
+ /**
+ * Holds the bean names of {@link ServiceBean} or {@link ServiceConfig}.
+ */
+ private Set<String> conflictedBeanNames = new LinkedHashSet<>();
+
+ @Override
+ public void postProcessMergedBeanDefinition(RootBeanDefinition beanDefinition, Class<?> beanType, String beanName) {
+ // Get raw bean type
+ Class<?> rawBeanType = getUserClass(beanType);
+ if (isAssignable(ServiceConfig.class, rawBeanType)) { // ServiceConfig type or sub-type
+ String interfaceName = (String) beanDefinition.getPropertyValues().get("interface");
+ String mappedBeanName = interfaceNamesToBeanNames.putIfAbsent(interfaceName, beanName);
+ // If mapped bean name exists and does not equal current bean name
+ if (mappedBeanName != null && !mappedBeanName.equals(beanName)) {
+ // conflictedBeanNames will record current bean name.
+ conflictedBeanNames.add(beanName);
+ }
+ }
+ }
+
+ @Override
+ public Object postProcessBeforeInitialization(Object bean, String beanName) throws BeansException {
+ if (conflictedBeanNames.contains(beanName) && bean instanceof ServiceConfig) {
+ ServiceConfig serviceConfig = (ServiceConfig) bean;
+ if (isConflictedServiceConfig(serviceConfig)) {
+ // Set id as the bean name
+ serviceConfig.setId(beanName);
+ }
+
+ }
+ return bean;
+ }
+
+ private boolean isConflictedServiceConfig(ServiceConfig serviceConfig) {
+ return Objects.equals(serviceConfig.getId(), serviceConfig.getInterface());
+ }
+
+ @Override
+ public Object postProcessAfterInitialization(Object bean, String beanName) throws BeansException {
+ return bean;
+ }
+
+ /**
+ * Keep the order being higher than {@link CommonAnnotationBeanPostProcessor#getOrder()} that is
+ * {@link Ordered#LOWEST_PRECEDENCE}
+ *
+ * @return {@link Ordered#LOWEST_PRECEDENCE} +1
+ */
+ @Override
+ public int getOrder() {
+ return LOWEST_PRECEDENCE + 1;
+ }
+
+ @Override
+ public void destroy() throws Exception {
+ interfaceNamesToBeanNames.clear();
+ conflictedBeanNames.clear();
+ }
+}