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

Changed the name of the package of the reflection.proto #1695

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

stefanadranca
Copy link
Collaborator

Motivation:

The name was not complete and GRPCurl's requests for the Reflection Service resulted in error.

Modifications:

Changed the name of the package, regenerated the Service code and changed the type of the variables accordingly, in the Service implementation and in the tests.

Result:

GRPCurl will be able to make requests to Servers implementing the Reflection Service.

Motivation:

The name was not complete and GRPCurl's requests for the Reflection Service
resulted in error.

Modifications:

Changed the name of the package, regenerated the Service code and
changed the type of the variables accordingly, in the Service implementation
and in the tests.

Result:

GRPCurl will be able to make requests to Servers implementing the
Reflection Service.
@stefanadranca stefanadranca requested a review from glbrntt October 30, 2023 19:24
@glbrntt glbrntt merged commit 10f3576 into grpc:main Oct 31, 2023
11 of 13 checks passed
@glbrntt glbrntt added the semver/none No version bump required. label Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants