proto - OpenGrok history log for /tonic/tonic-reflection/proto

Revision	Date	Author	Comments (<<< Hide modified files) (Show modified files >>>)
b29f4668	29-May-2024	katelyn martin <[email protected]>	tonic-reflection: �� use `v1` reflection protobuffer definitions (#1701) this fixes #1685. in penumbra-zone/penumbra#4392, we observed that tonic servers do not properly support reflection when ser tonic-reflection: �� use `v1` reflection protobuffer definitions (#1701) this fixes #1685. in penumbra-zone/penumbra#4392, we observed that tonic servers do not properly support reflection when servicing a request sent by recent versions of [`grpcurl`], after [v1.8.8] began using `grpc.reflection.v1.ServerReflection`. (see fullstorydev/grpcurl#407) these lead to an error regarding an unexpected status code, like this: ``` ❯ grpcurl --version grpcurl v1.9.1 ❯ grpcurl -vv grpc.testnet.penumbra.zone:443 list Failed to list services: rpc error: code = Unknown desc = unexpected HTTP status code received from server: 405 (Method Not Allowed); malformed header: missing HTTP content-type ``` this adds the v1 reflection definition to `tonic-reflection`, which was observed as fixing these issues for our gRPC endpoint. ### �� changes changes in this commet are as follows: * vendors the `v1` definition of [`reflection.proto`][proto]. * renames the (deprecated) `v1alpha` definition to `reflection_v1alpha.proto`. * `tonic_reflection::generated::grpc_reflection_v1` links to the generated Rust code (created by running `cargo run --package codegen`). * `tonic_reflection::generated::FILE_DESCRIPTOR_SET` is replaced by `tonic_reflection::generated::{FILE_DESCRIPTOR_SET_V1ALPHA, FILE_DESCRIPTOR_SET_V1}`. * `tonic_reflection::pb` now contains namespaced `tonic_reflection::pb::{v1alpha, v1}` submodules. (NB: this is a breaking change to the public `tonic-reflection` API.) * `tonic_reflection::server` is updated to use the generated `tonic_reflection::pb::v1` code. [v1.8.8]: https://github.com/fullstorydev/grpcurl/releases/tag/v1.8.8 [proto]: https://github.com/grpc/grpc/blob/master/src/proto/grpc/reflection/v1/reflection.proto [grpcurl]: https://github.com/fullstorydev/grpcurl --- fixes: #1685 x-ref: penumbra-zone/penumbra#4392 Co-authored-by: Conor Schaefer <[email protected]> show more ... /tonic/.github/assets/tonic-banner.svg /tonic/.github/workflows/CI.yml /tonic/CHANGELOG.md /tonic/CONTRIBUTING.md /tonic/Cargo.toml /tonic/README.md /tonic/SECURITY.md /tonic/codegen/Cargo.toml /tonic/codegen/src/main.rs /tonic/deny.toml /tonic/examples/Cargo.toml /tonic/examples/LICENSE /tonic/examples/README.md /tonic/examples/build.rs /tonic/examples/helloworld-tutorial.md /tonic/examples/proto/attrs/attrs.proto /tonic/examples/proto/unaryecho/echo.proto /tonic/examples/routeguide-tutorial.md /tonic/examples/src/authentication/client.rs /tonic/examples/src/authentication/server.rs /tonic/examples/src/autoreload/server.rs /tonic/examples/src/blocking/client.rs /tonic/examples/src/blocking/server.rs /tonic/examples/src/cancellation/client.rs /tonic/examples/src/cancellation/server.rs /tonic/examples/src/codec_buffers/client.rs /tonic/examples/src/codec_buffers/common.rs /tonic/examples/src/codec_buffers/server.rs /tonic/examples/src/compression/client.rs /tonic/examples/src/compression/server.rs /tonic/examples/src/dynamic/server.rs /tonic/examples/src/dynamic_load_balance/client.rs /tonic/examples/src/dynamic_load_balance/server.rs /tonic/examples/src/gcp/client.rs /tonic/examples/src/grpc-web/client.rs /tonic/examples/src/grpc-web/server.rs /tonic/examples/src/h2c/client.rs /tonic/examples/src/h2c/server.rs /tonic/examples/src/health/README.md /tonic/examples/src/interceptor/client.rs /tonic/examples/src/interceptor/server.rs /tonic/examples/src/json-codec/client.rs /tonic/examples/src/json-codec/common.rs /tonic/examples/src/json-codec/server.rs /tonic/examples/src/load_balance/client.rs /tonic/examples/src/load_balance/server.rs /tonic/examples/src/mock/mock.rs /tonic/examples/src/multiplex/client.rs /tonic/examples/src/multiplex/server.rs /tonic/examples/src/reflection/server.rs /tonic/examples/src/richer-error/client.rs /tonic/examples/src/richer-error/client_vec.rs /tonic/examples/src/richer-error/server.rs /tonic/examples/src/richer-error/server_vec.rs /tonic/examples/src/routeguide/client.rs /tonic/examples/src/routeguide/data.rs /tonic/examples/src/routeguide/server.rs /tonic/examples/src/streaming/client.rs /tonic/examples/src/streaming/server.rs /tonic/examples/src/tls/client.rs /tonic/examples/src/tls/server.rs /tonic/examples/src/tls_client_auth/client.rs /tonic/examples/src/tls_client_auth/server.rs /tonic/examples/src/tls_rustls/client.rs /tonic/examples/src/tls_rustls/server.rs /tonic/examples/src/tower/client.rs /tonic/examples/src/tower/server.rs /tonic/examples/src/tracing/client.rs /tonic/examples/src/uds/client.rs /tonic/examples/src/uds/server.rs /tonic/interop/Cargo.toml /tonic/interop/LICENSE /tonic/interop/src/bin/client.rs /tonic/interop/src/bin/server.rs /tonic/interop/src/client.rs /tonic/interop/src/lib.rs /tonic/interop/src/server.rs /tonic/prepare-release.sh /tonic/publish-release.sh /tonic/rustfmt.toml /tonic/tests/ambiguous_methods/Cargo.toml /tonic/tests/ambiguous_methods/LICENSE /tonic/tests/compression/Cargo.toml /tonic/tests/compression/LICENSE /tonic/tests/compression/build.rs /tonic/tests/compression/proto/test.proto /tonic/tests/compression/src/bidirectional_stream.rs /tonic/tests/compression/src/client_stream.rs /tonic/tests/compression/src/compressing_request.rs /tonic/tests/compression/src/compressing_response.rs /tonic/tests/compression/src/lib.rs /tonic/tests/compression/src/server_stream.rs /tonic/tests/compression/src/util.rs /tonic/tests/default_stubs/Cargo.toml /tonic/tests/default_stubs/build.rs /tonic/tests/default_stubs/proto/test.proto /tonic/tests/default_stubs/proto/test_default.proto /tonic/tests/default_stubs/src/lib.rs /tonic/tests/default_stubs/src/test_defaults.rs /tonic/tests/disable_comments/Cargo.toml /tonic/tests/disable_comments/build.rs /tonic/tests/disable_comments/proto/test.proto /tonic/tests/disable_comments/src/lib.rs /tonic/tests/disable_comments/tests/disable_comments.rs /tonic/tests/extern_path/my_application/Cargo.toml /tonic/tests/extern_path/my_application/LICENSE /tonic/tests/extern_path/uuid/Cargo.toml /tonic/tests/extern_path/uuid/LICENSE /tonic/tests/included_service/Cargo.toml /tonic/tests/included_service/LICENSE /tonic/tests/integration_tests/Cargo.toml /tonic/tests/integration_tests/LICENSE /tonic/tests/integration_tests/build.rs /tonic/tests/integration_tests/proto/stream.proto /tonic/tests/integration_tests/proto/test.proto /tonic/tests/integration_tests/src/lib.rs /tonic/tests/integration_tests/tests/client_layer.rs /tonic/tests/integration_tests/tests/complex_tower_middleware.rs /tonic/tests/integration_tests/tests/connect_info.rs /tonic/tests/integration_tests/tests/connection.rs /tonic/tests/integration_tests/tests/extensions.rs /tonic/tests/integration_tests/tests/interceptor.rs /tonic/tests/integration_tests/tests/max_message_size.rs /tonic/tests/integration_tests/tests/origin.rs /tonic/tests/integration_tests/tests/routes_builder.rs /tonic/tests/integration_tests/tests/status.rs /tonic/tests/integration_tests/tests/streams.rs /tonic/tests/integration_tests/tests/timeout.rs /tonic/tests/integration_tests/tests/user_agent.rs /tonic/tests/root-crate-path/Cargo.toml /tonic/tests/root-crate-path/LICENSE /tonic/tests/root-crate-path/build.rs /tonic/tests/root-crate-path/foo.proto /tonic/tests/root-crate-path/src/main.rs /tonic/tests/same_name/Cargo.toml /tonic/tests/same_name/LICENSE /tonic/tests/service_named_result/Cargo.toml /tonic/tests/service_named_result/LICENSE /tonic/tests/service_named_result/build.rs /tonic/tests/service_named_result/proto/result.proto /tonic/tests/service_named_result/src/lib.rs /tonic/tests/service_named_service/Cargo.toml /tonic/tests/service_named_service/LICENSE /tonic/tests/service_named_service/build.rs /tonic/tests/service_named_service/proto/foo.proto /tonic/tests/service_named_service/src/lib.rs /tonic/tests/stream_conflict/Cargo.toml /tonic/tests/stream_conflict/LICENSE /tonic/tests/stream_conflict/build.rs /tonic/tests/stream_conflict/proto/stream_conflict.proto /tonic/tests/stream_conflict/src/main.rs /tonic/tests/use_arc_self/Cargo.toml /tonic/tests/use_arc_self/LICENSE /tonic/tests/use_arc_self/build.rs /tonic/tests/use_arc_self/proto/test.proto /tonic/tests/use_arc_self/src/lib.rs /tonic/tests/wellknown-compiled/Cargo.toml /tonic/tests/wellknown-compiled/LICENSE /tonic/tests/wellknown-compiled/build.rs /tonic/tests/wellknown-compiled/proto/google.proto /tonic/tests/wellknown-compiled/proto/test.proto /tonic/tests/wellknown-compiled/src/lib.rs /tonic/tests/wellknown/Cargo.toml /tonic/tests/wellknown/LICENSE /tonic/tonic-build/Cargo.toml /tonic/tonic-build/LICENSE /tonic/tonic-build/README.md /tonic/tonic-build/src/client.rs /tonic/tonic-build/src/code_gen.rs /tonic/tonic-build/src/compile_settings.rs /tonic/tonic-build/src/lib.rs /tonic/tonic-build/src/manual.rs /tonic/tonic-build/src/prost.rs /tonic/tonic-build/src/server.rs /tonic/tonic-health/Cargo.toml /tonic/tonic-health/LICENSE /tonic/tonic-health/README.md /tonic/tonic-health/proto/health.proto /tonic/tonic-health/src/generated/grpc_health_v1.bin /tonic/tonic-health/src/generated/grpc_health_v1.rs /tonic/tonic-health/src/lib.rs /tonic/tonic-health/src/server.rs /tonic/tonic-reflection/Cargo.toml /tonic/tonic-reflection/LICENSE /tonic/tonic-reflection/README.md reflection_v1.proto reflection_v1alpha.proto /tonic/tonic-reflection/src/generated/grpc_reflection_v1.rs /tonic/tonic-reflection/src/generated/grpc_reflection_v1alpha.rs /tonic/tonic-reflection/src/generated/reflection_v1.bin /tonic/tonic-reflection/src/generated/reflection_v1alpha1.bin /tonic/tonic-reflection/src/lib.rs /tonic/tonic-reflection/src/server.rs /tonic/tonic-reflection/tests/server.rs /tonic/tonic-types/Cargo.toml /tonic/tonic-types/LICENSE /tonic/tonic-types/README.md /tonic/tonic-types/proto/error_details.proto /tonic/tonic-types/src/generated/google_rpc.rs /tonic/tonic-types/src/generated/types.bin /tonic/tonic-types/src/lib.rs /tonic/tonic-types/src/richer_error/error_details/mod.rs /tonic/tonic-types/src/richer_error/error_details/vec.rs /tonic/tonic-types/src/richer_error/mod.rs /tonic/tonic-types/src/richer_error/std_messages/bad_request.rs /tonic/tonic-types/src/richer_error/std_messages/debug_info.rs /tonic/tonic-types/src/richer_error/std_messages/error_info.rs /tonic/tonic-types/src/richer_error/std_messages/help.rs /tonic/tonic-types/src/richer_error/std_messages/loc_message.rs /tonic/tonic-types/src/richer_error/std_messages/mod.rs /tonic/tonic-types/src/richer_error/std_messages/prec_failure.rs /tonic/tonic-types/src/richer_error/std_messages/quota_failure.rs /tonic/tonic-types/src/richer_error/std_messages/request_info.rs /tonic/tonic-types/src/richer_error/std_messages/resource_info.rs /tonic/tonic-types/src/richer_error/std_messages/retry_info.rs /tonic/tonic-web/Cargo.toml /tonic/tonic-web/LICENSE /tonic/tonic-web/README.md /tonic/tonic-web/src/call.rs /tonic/tonic-web/src/client.rs /tonic/tonic-web/src/layer.rs /tonic/tonic-web/src/lib.rs /tonic/tonic-web/src/service.rs /tonic/tonic-web/tests/incomplete-buf-bug.bin /tonic/tonic-web/tests/integration/Cargo.toml /tonic/tonic-web/tests/integration/LICENSE /tonic/tonic-web/tests/integration/build.rs /tonic/tonic-web/tests/integration/proto/test.proto /tonic/tonic-web/tests/integration/src/lib.rs /tonic/tonic-web/tests/integration/tests/grpc.rs /tonic/tonic-web/tests/integration/tests/grpc_web.rs /tonic/tonic/Cargo.toml /tonic/tonic/LICENSE /tonic/tonic/benches-disabled/benchmarks/compiled_protos/helloworld.rs /tonic/tonic/benches/decode.rs /tonic/tonic/src/body.rs /tonic/tonic/src/client/grpc.rs /tonic/tonic/src/client/service.rs /tonic/tonic/src/codec/buffer.rs /tonic/tonic/src/codec/compression.rs /tonic/tonic/src/codec/decode.rs /tonic/tonic/src/codec/encode.rs /tonic/tonic/src/codec/mod.rs /tonic/tonic/src/codec/prost.rs /tonic/tonic/src/codegen.rs /tonic/tonic/src/extensions.rs /tonic/tonic/src/lib.rs /tonic/tonic/src/macros.rs /tonic/tonic/src/metadata/encoding.rs /tonic/tonic/src/metadata/key.rs /tonic/tonic/src/metadata/map.rs /tonic/tonic/src/metadata/mod.rs /tonic/tonic/src/metadata/value.rs /tonic/tonic/src/request.rs /tonic/tonic/src/response.rs /tonic/tonic/src/server/grpc.rs /tonic/tonic/src/server/mod.rs /tonic/tonic/src/server/service.rs /tonic/tonic/src/service/interceptor.rs /tonic/tonic/src/service/mod.rs /tonic/tonic/src/status.rs /tonic/tonic/src/transport/channel/endpoint.rs /tonic/tonic/src/transport/channel/mod.rs /tonic/tonic/src/transport/channel/tls.rs /tonic/tonic/src/transport/error.rs /tonic/tonic/src/transport/mod.rs /tonic/tonic/src/transport/server/conn.rs /tonic/tonic/src/transport/server/incoming.rs /tonic/tonic/src/transport/server/mod.rs /tonic/tonic/src/transport/server/recover_error.rs /tonic/tonic/src/transport/server/tls.rs /tonic/tonic/src/transport/server/unix.rs /tonic/tonic/src/transport/service/add_origin.rs /tonic/tonic/src/transport/service/connection.rs /tonic/tonic/src/transport/service/connector.rs /tonic/tonic/src/transport/service/discover.rs /tonic/tonic/src/transport/service/executor.rs /tonic/tonic/src/transport/service/grpc_timeout.rs /tonic/tonic/src/transport/service/io.rs /tonic/tonic/src/transport/service/mod.rs /tonic/tonic/src/transport/service/reconnect.rs /tonic/tonic/src/transport/service/router.rs /tonic/tonic/src/transport/service/tls.rs /tonic/tonic/src/transport/tls.rs /tonic/tonic/src/util.rs
c54f2472	16-Feb-2021	James Nugent <[email protected]>	feat: Implement gRPC Reflection Service (#340) Co-authored-by: Samani G. Gikandi <[email protected]> /tonic/.github/ISSUE_TEMPLATE/bug_report.md /tonic/.github/ISSUE_TEMPLATE/feature_request.md /tonic/.github/PULL_REQUEST_TEMPLATE.md /tonic/.github/assets/tonic-banner.svg /tonic/.github/assets/tonic-docs.png /tonic/.github/workflows/CI.yml /tonic/.gitignore /tonic/CHANGELOG.md /tonic/CONTRIBUTING.md /tonic/Cargo.toml /tonic/LICENSE /tonic/README.md /tonic/deny.toml /tonic/examples/Cargo.toml /tonic/examples/README.md /tonic/examples/build.rs /tonic/examples/data/gcp/roots.pem /tonic/examples/data/route_guide_db.json /tonic/examples/data/tls/ca.pem /tonic/examples/data/tls/client1.key /tonic/examples/data/tls/client1.pem /tonic/examples/data/tls/client2.key /tonic/examples/data/tls/client2.pem /tonic/examples/data/tls/client_ca.pem /tonic/examples/data/tls/server.key /tonic/examples/data/tls/server.pem /tonic/examples/helloworld-tutorial.md /tonic/examples/proto/echo/echo.proto /tonic/examples/proto/googleapis/google/api/annotations.proto /tonic/examples/proto/googleapis/google/api/client.proto /tonic/examples/proto/googleapis/google/api/field_behavior.proto /tonic/examples/proto/googleapis/google/api/http.proto /tonic/examples/proto/googleapis/google/api/resource.proto /tonic/examples/proto/googleapis/google/pubsub/v1/pubsub.proto /tonic/examples/proto/googleapis/google/pubsub/v1/schema.proto /tonic/examples/proto/helloworld/helloworld.proto /tonic/examples/proto/routeguide/route_guide.proto /tonic/examples/routeguide-tutorial.md /tonic/examples/src/authentication/client.rs /tonic/examples/src/authentication/server.rs /tonic/examples/src/autoreload/server.rs /tonic/examples/src/blocking/client.rs /tonic/examples/src/dynamic_load_balance/client.rs /tonic/examples/src/dynamic_load_balance/server.rs /tonic/examples/src/gcp/README.md /tonic/examples/src/gcp/client.rs /tonic/examples/src/health/server.rs /tonic/examples/src/helloworld/client.rs /tonic/examples/src/helloworld/server.rs /tonic/examples/src/helloworld/server_blocking.rs /tonic/examples/src/hyper_warp/client.rs /tonic/examples/src/hyper_warp/server.rs /tonic/examples/src/hyper_warp_multiplex/client.rs /tonic/examples/src/hyper_warp_multiplex/server.rs /tonic/examples/src/interceptor/client.rs /tonic/examples/src/interceptor/server.rs /tonic/examples/src/load_balance/client.rs /tonic/examples/src/load_balance/server.rs /tonic/examples/src/multiplex/client.rs /tonic/examples/src/multiplex/server.rs /tonic/examples/src/optional/server.rs /tonic/examples/src/reflection/server.rs /tonic/examples/src/routeguide/client.rs /tonic/examples/src/routeguide/data.rs /tonic/examples/src/routeguide/server.rs /tonic/examples/src/timeout/client.rs /tonic/examples/src/timeout/server.rs /tonic/examples/src/tls/client.rs /tonic/examples/src/tls/server.rs /tonic/examples/src/tls_client_auth/client.rs /tonic/examples/src/tls_client_auth/server.rs /tonic/examples/src/tower/client.rs /tonic/examples/src/tower/server.rs /tonic/examples/src/tracing/client.rs /tonic/examples/src/tracing/server.rs /tonic/examples/src/uds/client.rs /tonic/examples/src/uds/server.rs /tonic/interop/Cargo.toml /tonic/interop/bin/client_darwin_amd64 /tonic/interop/bin/client_linux_amd64 /tonic/interop/bin/client_windows_amd64.exe /tonic/interop/bin/server_darwin_amd64 /tonic/interop/bin/server_linux_amd64 /tonic/interop/bin/server_windows_amd64.exe /tonic/interop/build.rs /tonic/interop/data/README.md /tonic/interop/data/ca.pem /tonic/interop/data/cert-generator/.gitignore /tonic/interop/data/cert-generator/ca.tf /tonic/interop/data/cert-generator/server_certs.tf /tonic/interop/data/server1.key /tonic/interop/data/server1.pem /tonic/interop/proto/grpc/testing/empty.proto /tonic/interop/proto/grpc/testing/messages.proto /tonic/interop/proto/grpc/testing/test.proto /tonic/interop/src/bin/client.rs /tonic/interop/src/bin/server.rs /tonic/interop/src/client.rs /tonic/interop/src/lib.rs /tonic/interop/src/server.rs /tonic/interop/test.sh /tonic/interop/update_binaries.sh /tonic/rustfmt.toml /tonic/tests/ambiguous_methods/Cargo.toml /tonic/tests/ambiguous_methods/build.rs /tonic/tests/ambiguous_methods/proto/ambiguous_methods.proto /tonic/tests/ambiguous_methods/src/main.rs /tonic/tests/extern_path/my_application/Cargo.toml /tonic/tests/extern_path/my_application/build.rs /tonic/tests/extern_path/my_application/src/main.rs /tonic/tests/extern_path/proto/my_application/service.proto /tonic/tests/extern_path/proto/uuid/uuid.proto /tonic/tests/extern_path/uuid/Cargo.toml /tonic/tests/extern_path/uuid/build.rs /tonic/tests/extern_path/uuid/src/lib.rs /tonic/tests/included_service/Cargo.toml /tonic/tests/included_service/build.rs /tonic/tests/included_service/proto/includee.proto /tonic/tests/included_service/proto/includer.proto /tonic/tests/included_service/src/lib.rs /tonic/tests/integration_tests/Cargo.toml /tonic/tests/integration_tests/build.rs /tonic/tests/integration_tests/proto/test.proto /tonic/tests/integration_tests/src/lib.rs /tonic/tests/integration_tests/tests/connection.rs /tonic/tests/integration_tests/tests/status.rs /tonic/tests/integration_tests/tests/user_agent.rs /tonic/tests/same_name/Cargo.toml /tonic/tests/same_name/build.rs /tonic/tests/same_name/proto/foo.proto /tonic/tests/same_name/src/lib.rs /tonic/tests/wellknown/Cargo.toml /tonic/tests/wellknown/build.rs /tonic/tests/wellknown/proto/wellknown.proto /tonic/tests/wellknown/src/lib.rs /tonic/tonic-build/Cargo.toml /tonic/tonic-build/README.md /tonic/tonic-build/src/client.rs /tonic/tonic-build/src/lib.rs /tonic/tonic-build/src/prost.rs /tonic/tonic-build/src/server.rs /tonic/tonic-health/Cargo.toml /tonic/tonic-health/README.md /tonic/tonic-health/build.rs /tonic/tonic-health/proto/health.proto /tonic/tonic-health/src/lib.rs /tonic/tonic-health/src/server.rs /tonic/tonic-reflection/Cargo.toml /tonic/tonic-reflection/build.rs reflection.proto /tonic/tonic-reflection/src/lib.rs /tonic/tonic-reflection/src/server.rs /tonic/tonic-reflection/tests/proto/grpc.reflection.v1alpha.rs /tonic/tonic-reflection/tests/proto/mod.rs /tonic/tonic-reflection/tests/server.rs /tonic/tonic-types/Cargo.toml /tonic/tonic-types/build.rs /tonic/tonic-types/proto/status.proto /tonic/tonic-types/src/lib.rs /tonic/tonic/Cargo.toml /tonic/tonic/benches-disabled/README.md /tonic/tonic/benches-disabled/bench_main.rs /tonic/tonic/benches-disabled/benchmarks/compiled_protos/diverse_types.rs /tonic/tonic/benches-disabled/benchmarks/compiled_protos/helloworld.rs /tonic/tonic/benches-disabled/benchmarks/compiled_protos/mod.rs /tonic/tonic/benches-disabled/benchmarks/mod.rs /tonic/tonic/benches-disabled/benchmarks/request_response.rs /tonic/tonic/benches-disabled/benchmarks/request_response_diverse_types.rs /tonic/tonic/benches-disabled/benchmarks/utils.rs /tonic/tonic/benches-disabled/proto/diverse_types/diverse_types.proto /tonic/tonic/benches-disabled/proto/helloworld/helloworld.proto /tonic/tonic/benches/decode.rs /tonic/tonic/src/body.rs /tonic/tonic/src/client/grpc.rs /tonic/tonic/src/client/mod.rs /tonic/tonic/src/client/service.rs /tonic/tonic/src/codec/buffer.rs /tonic/tonic/src/codec/decode.rs /tonic/tonic/src/codec/encode.rs /tonic/tonic/src/codec/mod.rs /tonic/tonic/src/codec/prost.rs /tonic/tonic/src/codegen.rs /tonic/tonic/src/interceptor.rs /tonic/tonic/src/lib.rs /tonic/tonic/src/macros.rs /tonic/tonic/src/metadata/encoding.rs /tonic/tonic/src/metadata/key.rs /tonic/tonic/src/metadata/map.rs /tonic/tonic/src/metadata/mod.rs /tonic/tonic/src/metadata/value.rs /tonic/tonic/src/request.rs /tonic/tonic/src/response.rs /tonic/tonic/src/server/grpc.rs /tonic/tonic/src/server/mod.rs /tonic/tonic/src/server/service.rs /tonic/tonic/src/status.rs /tonic/tonic/src/transport/channel/endpoint.rs /tonic/tonic/src/transport/channel/mod.rs /tonic/tonic/src/transport/channel/tls.rs /tonic/tonic/src/transport/error.rs /tonic/tonic/src/transport/mod.rs /tonic/tonic/src/transport/server/conn.rs /tonic/tonic/src/transport/server/incoming.rs /tonic/tonic/src/transport/server/mod.rs /tonic/tonic/src/transport/server/tls.rs /tonic/tonic/src/transport/service/add_origin.rs /tonic/tonic/src/transport/service/connection.rs /tonic/tonic/src/transport/service/connector.rs /tonic/tonic/src/transport/service/discover.rs /tonic/tonic/src/transport/service/io.rs /tonic/tonic/src/transport/service/layer.rs /tonic/tonic/src/transport/service/mod.rs /tonic/tonic/src/transport/service/reconnect.rs /tonic/tonic/src/transport/service/router.rs /tonic/tonic/src/transport/service/tls.rs /tonic/tonic/src/transport/service/user_agent.rs /tonic/tonic/src/transport/tls.rs

Revision

Date

Author

Comments

(<<< Hide modified files)

(Show modified files >>>)

b29f4668

29-May-2024

katelyn martin <[email protected]>

tonic-reflection: �� use `v1` reflection protobuffer definitions (#1701)

this fixes #1685.

in penumbra-zone/penumbra#4392, we observed that tonic servers do not
properly support reflection when servicing a request sent by recent
versions of [`grpcurl`], after [v1.8.8] began using
`grpc.reflection.v1.ServerReflection`. (see fullstorydev/grpcurl#407)

these lead to an error regarding an unexpected status code, like this:

```
❯ grpcurl --version
grpcurl v1.9.1

❯ grpcurl -vv grpc.testnet.penumbra.zone:443 list
Failed to list services: rpc error: code = Unknown desc = unexpected HTTP status code received from server: 405 (Method Not Allowed); malformed header: missing HTTP content-type
```

this adds the v1 reflection definition to `tonic-reflection`, which was
observed as fixing these issues for our gRPC endpoint.

### �� changes

changes in this commet are as follows:

* vendors the `v1` definition of [`reflection.proto`][proto].

* renames the (deprecated) `v1alpha` definition to
`reflection_v1alpha.proto`.

* `tonic_reflection::generated::grpc_reflection_v1` links to the
generated Rust code (created by running `cargo run --package codegen`).

* `tonic_reflection::generated::FILE_DESCRIPTOR_SET` is replaced by
`tonic_reflection::generated::{FILE_DESCRIPTOR_SET_V1ALPHA, FILE_DESCRIPTOR_SET_V1}`.

* `tonic_reflection::pb` now contains namespaced
`tonic_reflection::pb::{v1alpha, v1}` submodules. (**NB: this is a
breaking change to the public `tonic-reflection` API.**)

* `tonic_reflection::server` is updated to use the generated
`tonic_reflection::pb::v1` code.

[v1.8.8]: https://github.com/fullstorydev/grpcurl/releases/tag/v1.8.8
[proto]: https://github.com/grpc/grpc/blob/master/src/proto/grpc/reflection/v1/reflection.proto
[grpcurl]: https://github.com/fullstorydev/grpcurl

---

fixes: #1685
x-ref: penumbra-zone/penumbra#4392

Co-authored-by: Conor Schaefer <[email protected]>

c54f2472

16-Feb-2021

James Nugent <[email protected]>

feat: Implement gRPC Reflection Service (#340)

Co-authored-by: Samani G. Gikandi <[email protected]>