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

Aleks+murisi/upgrade v0.22.0 #10

Closed
wants to merge 3 commits into from

Conversation

murisi
Copy link
Contributor

@murisi murisi commented Sep 15, 2023

Made changes to the Ledger app to support Namada v0.22.0 (with multisig adjustments). More specifically, we did the following:

  • Upgraded the signature section format to one that allows multiple signatures over a set of hashes and made corresponding changes to how signature sections are hashed
  • Modified the Ledger app's response to include the set of section indices signed over, because the Namada web interface requires this information to construct a valid signature section
  • Implemented a parser for signature sections so that the wrapper signature can sign over these as well (under the condition that its hashes have already been validated). This turned out to be most straightforward way to achieve multisig.
  • Upgraded the parsers of the various transaction types in order to be compatible with Namada v0.22.0. Introduced a new transaction type called Unjail Validator during this process. Partially merged in IBC support from the ibc_support branch.
  • Updated parsing and signing test vectors and WASM code hashes.

@murisi murisi force-pushed the aleks+murisi/upgrade-v0.22.0 branch from e547c17 to 595c0a8 Compare September 15, 2023 07:41
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

Successfully merging this pull request may close these issues.

2 participants