Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[type:fix] Fix dubbo3.0 cannot get service provider information #5708

Open
wants to merge 63 commits into
base: master
Choose a base branch
from

Conversation

hujiatai
Copy link

@hujiatai hujiatai commented Oct 14, 2024

该处解决dubbo3.0后实例服务发现时,当默认使用注册中心的地址作为元数据中心时,无法使用到默认的元数据中心获取到接口的提供者,导致rpc失败
修改此次后当没有配置元数据中心时会使用注册中心的地址作为元数据中心
2.6.1版本启用dubbo插件时没有配置元数据中心的地址配置,后续应前后端都允许元数据中心的配置

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

该处解决dubbo3.0后实例服务发现时,当默认使用注册中心的地址作为元数据中心时,无法使用到默认的元数据中心获取到接口的提供者,导致rpc失败
 修改此次后当没有配置元数据中心时会使用注册中心的地址作为元数据中心
 2.6.1版本启用dubbo插件时没有配置元数据中心的地址配置,后续应前后端都允许元数据中心的配置
@moremind
Copy link
Member

pls use english, after you modify, please contact me with wechat: yijia1949

…sed to expose the registration, the provider of interface service cannot be obtained

When solving the instance service discovery after dubbo3.0, when the default address of the registration center is used as the metadata center, the default metadata center cannot be used to obtain the interface provider, resulting in rpc failure
 When no metadata center is configured after this modification, the address of the registry will be used as the metadata center
The address configuration of the metadata center is not configured when the dubbo plug-in is enabled in version 2.6.1, and the configuration of the metadata center should be allowed in both the front-end and back-end in the future
@moremind moremind changed the title [type:fix] dubbo插件在dubbo 3.0时,只使用实例服务暴露注册时,无法获取到接口服务的提供者 [type:fix] Fix dubbo3.0 cannot get service provider information Oct 15, 2024
@hujiatai hujiatai requested a review from moremind October 15, 2024 02:46
@moremind
Copy link
Member

moremind commented Nov 5, 2024

not right, you should run shenyu-integrate-combination module

@hujiatai
Copy link
Author

hujiatai commented Nov 5, 2024 via email

@hujiatai
Copy link
Author

hujiatai commented Nov 6, 2024

ruined, test pass Uploading d4c9deab7b9fd0e35e18e6ff01fc731e.jpg…
IMG_8265 JPG

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants