fullstorydev / grpcurl

Like cURL, but for gRPC: Command-line tool for interacting with gRPC servers
MIT License
10.35k stars 497 forks source link

Use latest protoreflect to fix some bugs #453

Closed jhump closed 2 months ago

jhump commented 2 months ago

The use of AllowMissingFileDescriptors should resolve #451 (at least in many cases) as well as similar issues that were reported starting in v1.8.8, such as #423 and #432.

This also includes a fix for #417.

The protoreflect repo (as well as recent versions of grpc-go) now requires Go 1.19, so I've updated the go.mod file and the CI configs.

FWIW, grpcui will want basically the same changes (since similar things were reported there, like https://github.com/fullstorydev/grpcui/issues/279). It would be nice to cut a release of grpcurl after this is merged, and then pull that new grpcurl version into grpcui and cut a release there as well.

jhump commented 2 months ago

FYI @gpassini, @dragonsinth