Copilot commented on code in PR #135:
URL:
https://github.com/apache/dubbo-go-pixiu-samples/pull/135#discussion_r3146706563
##########
dubbogo/simple/proxy/pixiu/api_config.yaml:
##########
@@ -45,4 +43,4 @@ resources:
mapTo: opt.group
- name: queryStrings.version
mapTo: opt.version
Review Comment:
The route path still includes `:application`
(`/api/v1/test-dubbo/:application/:interface`), but the mapping for
`uri.application -> opt.application` was removed. This makes the `application`
path segment unused and may break Dubbo proxy resolution if `opt.application`
is expected (other sample configs still map it). Either restore the mapping for
`uri.application`, or remove `:application` from the path if it’s no longer
needed.
--
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]
