-
Notifications
You must be signed in to change notification settings - Fork 26.5k
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
[Bug] The triple service written using proto does not support capitalization of service methods when called on rest #15291
Comments
method name in proto is capitalized (apache#15291)
methods name |
you might refer to this proto: https://github.com/apache/dubbo/blob/3.3/dubbo-rpc/dubbo-rpc-triple/src/main/proto/health.proto
|
i tested on current dubbo 3.3 branch and didn't find the issue.
(2) add plugins, set os.detected.classifier to windows-x86_64 since i test it on windows.
(2) add management and server port configurations,
|
Pre-check
Search before asking
Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
Dubbo 3.3.4、 jdk 17
Steps to reproduce this issue
use dubbo-samples-triple-grpc, the old protobuf file is follow:
compile and run
Requests starting with uppercase or lowercase letters, call on http
The first http request detail (rpc method name is uppercase):
Error details:
The second http request detail (rpc method name first letter is uppercase):
Error detials:
What you expected to happen
Call 'localhost: 50053/org.apache.dubbo.samples.tri.grpc.Greeter/Greet'
Or 'localhost: 50053/org.apache.dubbo.samples.tri.grpc.Greeter/greet' via HTTP succeeded, do not all fail
Anything else
General proto file specifications, The first character of the RPC method name is capitalized, and we have a large number of proto files with history that use RPC method names starting with uppercase characters
Are you willing to submit a pull request to fix on your own?
Code of Conduct
The text was updated successfully, but these errors were encountered: