feat(grpc): Add ABCIQuery gRPC endpoint and register in BaseApp

[ChristianBorst]

Description

Related To: #11012
Potentially Closes: #7036

Exposes the ABCI Query method via gRPC, useful for obtaining proofs of application state without occupying Tendermint JSON-RPC bandwith. This is a pressing need with slow Tendermint JSON-RPC endpoints being used (and overloaded) by IBC relayers, but is also useful as a general purpose proven query source, enabling gRPC-based trustless clients.

---

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

• included the correct type prefix in the PR title
• added ! to the type prefix if API or client breaking change
• targeted the correct branch (see PR Targeting)
• provided a link to the relevant issue or specification
• followed the guidelines for building modules
• included the necessary unit and integration tests
• added a changelog entry to CHANGELOG.md
• included comments for documenting Go code
• updated the relevant documentation or specification
• reviewed "Files changed" and left comments if necessary
• confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.

I have...

• confirmed the correct type prefix in the PR title
• confirmed ! in the type prefix if API or client breaking change
• confirmed all author checklist items have been addressed
• reviewed state machine logic
• reviewed API design and naming
• reviewed documentation is accurate
• reviewed tests and test coverage
• manually tested (if applicable)

[aaronc]

Thanks @ChristianBorst! I know it's annoying but we would prefer not to depend on tendermint types here. Can you create unique types in the query.proto file?

[ChristianBorst]

Thanks @ChristianBorst! I know it's annoying but we would prefer not to depend on tendermint types here. Can you create unique types in the query.proto file?

Sure thing!

[ChristianBorst]

Thanks @ChristianBorst! I know it's annoying but we would prefer not to depend on tendermint types here. Can you create unique types in the query.proto file?

Done @aaronc, I hope this use of tendermint's ProofOps is alright, let me know if not.

[aaronc]

This shouldn't go in types/. We need to find another package. Also where is the implementation?

[amaury1093]

Hey @ChristianBorst, thanks! We already have a Tendermint proxy service in proto/cosmos/base/tendermint/v1beta1/query.proto. What do you think of putting this there?

[ChristianBorst]

I'm fine with that, and it should resolve @aaronc's issue with having the query endpoint somewhere other than the root of types/.

[ChristianBorst]

This shouldn't go in types/. We need to find another package. Also where is the implementation?

The implementation is the app itself, I wrapped the app with Tendermint's ABCI Application wrapper to make the signature work. This forwards the request to the same source Tendermint uses, completely bypassing Tendermint.

[tac0turtle]

@ChristianBorst id love to land this in the sdk for relayers as a short term solution. Do you want to push it to the finish line?

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[charleenfei]

bumping this pr as it looks like its gone stale

[alexanderbez]

@ChristianBorst would you like me to take this over?

[jackzampolin]

This is needed for us to fully move to gRPC on the go relayer. Excited to see this in!

[alexanderbez]

Ok, I'll have this wrapped up by EOD.