| #
6856ffe3 |
| 22-Oct-2024 |
tottoto <[email protected]> |
chore(codegen): Remove unnecessary collecting iterator (#2018)
|
| #
1470dfb9 |
| 27-Sep-2024 |
tottoto <[email protected]> |
chore: Generate file descriptor set constant directly (#1956)
|
|
Revision tags: v0.12.3 |
|
| #
6d93c1d0 |
| 23-Sep-2024 |
tottoto <[email protected]> |
chore: Make codegen independent from protoc (#1953)
|
| #
e4c71b8e |
| 04-Sep-2024 |
tottoto <[email protected]> |
feat(build): Rename compile method with compile_protos (#1920)
|
|
Revision tags: v0.12.2, v0.12.1, v0.12.0 |
|
| #
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 ...
|
|
Revision tags: v0.11.0 |
|
| #
522a8d74 |
| 13-Nov-2023 |
tottoto <[email protected]> |
chore: Fix clippy warning (#1500)
Co-authored-by: Lucio Franco <[email protected]>
|
|
Revision tags: v0.10.2, v0.10.1, v0.10.0 |
|
| #
7b2b0eea |
| 28-Aug-2023 |
tottoto <[email protected]> |
chore: Add generated code to module tree (#1480)
|
| #
44aa46db |
| 25-Aug-2023 |
tottoto <[email protected]> |
chore: Move codegen to separate crate (#1467)
|