gRPC: fix server reflection (backport #8945)

[mergify]

This is an automatic backport of pull request #8945 done by Mergify.
Cherry-pick of ac48ffe has failed:

On branch mergify/bp/release/v0.42.x/pr-8945
Your branch is up to date with 'origin/release/v0.42.x'.

You are currently cherry-picking commit ac48ffe74.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/run-node/interact-node.md
	new file:   server/grpc/gogoreflection/doc.go
	new file:   server/grpc/gogoreflection/fix_registration.go
	new file:   server/grpc/gogoreflection/fix_registration_test.go
	new file:   server/grpc/gogoreflection/serverreflection.go
	modified:   server/grpc/server.go
	modified:   server/grpc/server_test.go

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   CHANGELOG.md
	both modified:   go.mod
	both modified:   go.sum

To fix up this pull request, you can check it out locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

---

Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

• @Mergifyio refresh will re-evaluate the rules
• @Mergifyio rebase will rebase this PR on its base branch
• @Mergifyio update will merge the base branch into this PR
• @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

• look at your merge queues
• generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

[amaury1093]

Hmm, there's a weird error:

            	            	Symbol not found: ibc.core.connection.v1.Query
            	            	caused by: File not found: confio/proofs.proto

i can take a look (unless if @fdymylja you have an immediate idea what's happening?)

[fdymylja]

@AmauryM https://github.com/confio/ics23/blob/master/go/proofs.pb.go#L1158

proofs registers itself as proof.proto

but is imported most likely by IBC as confio/proofs.proto

which causes reflectiont to fail because the file in the registry is registered with a different path compared to how descriptors import it as.

[amaury1093]

This is unfortunate (ref: #9253 (comment)). In 07cf792 I moved proofs.proto to the third_party/proto root folder.

Does this workaround work for you @fdymylja?

Is the correct way of solving this to create a PR on https://github.com/confio/ics23/ to move the proofs.proto file there into a confio subfolder?

[amaury1093]

An alternative is just to not backport this fix...

[amaury1093]

Discussed with @robert-zaremba, let's just not backport this PR, and ask people to use 0.43 if they want server reflection.

[robert-zaremba]

We skip this PR. The fix here is only gRPC related. It doesn't break any logical flow nor existing tools nor is related to security. This feature did not work in the past.

gRPC is reflection is correctly implemented in v0.43. We just released beta (v0.43-beta), and RC is nearby.