Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove proof paths from IBC queries #7723

Closed
4 tasks
colin-axner opened this issue Oct 29, 2020 · 0 comments · Fixed by #7725
Closed
4 tasks

Remove proof paths from IBC queries #7723

colin-axner opened this issue Oct 29, 2020 · 0 comments · Fixed by #7725
Assignees
@colin-axner
Labels
T: Client UX Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity.
Milestone
IBC 1.0 Implement...

Comments

@colin-axner
Copy link
Contributor

Summary

IBC queries currently return a proof path, but this information is unnecessary since no message ever takes in this information. The blockchain must construct the paths in order to verify relayer submitted proofs. We can remove this field.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@colin-axner colin-axner added x/ibc Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity. T: Client UX labels Oct 29, 2020
@colin-axner colin-axner self-assigned this Oct 29, 2020
@colin-axner colin-axner mentioned this issue Oct 29, 2020
Merged
9 tasks
@mergify mergify bot closed this as completed in #7725 Oct 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@colin-axner colin-axner

Labels
T: Client UX Type: Code Hygiene General cleanup and restructuring of code to provide clarity, flexibility, and modularity.
Projects
None yet
Milestone
IBC 1.0 Implementation Audit
Development

Successfully merging a pull request may close this issue.

remove proof path from IBC queries cosmos/cosmos-sdk
1 participant
@colin-axner