NeverENG opened a new pull request, #3331:
URL: https://github.com/apache/dubbo-go/pull/3331
### Description
Fixes #issue3330
## What is the purpose of the change
Fix Apollo config center address handling when the Apollo server is
deployed under a context path, for example:
```yaml
config-center:
protocol: apollo
address: 127.0.0.1:8080/config
Previously, dubbo-go parsed /config into url.Path, but the Apollo config
center only used url.Location when building the agollo server address. As a
result, the final Apollo address became http://127.0.0.1:8080, and the conte
xt path was dropped.
## Brief changelog
- Preserve url.Path when building the Apollo config center address.
- Keep existing behavior unchanged for Apollo addresses without a context
path.
## Verifying this change
go test ./config_center/apollo
### Checklist
- [x] I confirm the target branch is `develop`
- [x] Code has passed local testing
- [x] I have added tests that prove my fix is effective or that my feature
works
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
