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.

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

Implement query_header_at_height() #336

Closed
5 tasks done
ancazamfir opened this issue Oct 21, 2020 · 1 comment
Closed
5 tasks done

Implement query_header_at_height() #336

ancazamfir opened this issue Oct 21, 2020 · 1 comment
Assignees
Milestone

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented Oct 21, 2020

Crate

relayer

Summary

Implement https://github.com/informalsystems/ibc-rs/blob/ab6cac1e3dec846b9fb5c2fee0deb6a634f0f290/relayer/src/chain.rs#L112

Problem Definition

Missing implementation. To build a MsgCreateClient the relayer needs a recent LigthtBlock. For MsgUpdateClient we would need the get_minimal_set() but the query_header_at_height() above would be great if it's faster.

Proposal

  • Initially, retrieve the header using plain RPC calls
  • At a later stage, retrieve and verify the header via the light client

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ancazamfir ancazamfir changed the title Implement Implement query_header_at_height() Oct 21, 2020
@ancazamfir ancazamfir modified the milestones: v0.0.5, v0.0.6 Oct 21, 2020
@romac romac added client I: logic Internal: related to the relaying logic and removed I: logic Internal: related to the relaying logic labels Oct 27, 2020
@adizere adizere mentioned this issue Nov 9, 2020
42 tasks
@adizere adizere modified the milestones: v0.0.6, v0.0.7 Dec 23, 2020
@ancazamfir
Copy link
Collaborator Author

Closed via #364 (see build_header() in chain.rs)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants