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

Remove a patch that overrides ValidateSelfClient #24

Open
bluele opened this issue Jan 27, 2022 · 0 comments
Open

Remove a patch that overrides ValidateSelfClient #24

bluele opened this issue Jan 27, 2022 · 0 comments

Comments

@bluele
Copy link
Member

bluele commented Jan 27, 2022

Currently, in a chain that has a client, which uses the proxy, the self clientState needs to be unwrapped to the underlying client state with ValidateSelfClient.

This can be fixed as follows:

  1. pass the actual self client state ConnOpen* handlers instead (also, we need to fix the relayer)
  2. VerifyClientState of the proxy client wraps the given clientState with MultiV client

This and moving GetTimestampAtHeight function into ClientState will achieve that there is no patch for the core modules.

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

1 participant