-
Notifications
You must be signed in to change notification settings - Fork 666
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 CheckMetadataHash
extension (#4274)
#4619
Conversation
This implements the `CheckMetadataHash` extension as described in [RFC78](https://polkadot-fellows.github.io/RFCs/approved/0078-merkleized-metadata.html). Besides the signed extension, the `substrate-wasm-builder` is extended to support generating the metadata-hash. Closes: #291 --------- Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io> Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com> Co-authored-by: Liam Aharon <liam.aharon@hotmail.com> Co-authored-by: Kian Paimani <5588131+kianenigma@users.noreply.github.com>
The CI pipeline was cancelled due to failure one of the required jobs. |
//! trust the metadata to be correct. It is very important for the metadata to be correct or | ||
//! otherwise an attacker could change them in a way that the offline wallet decodes a transaction | ||
//! in a different way than what it will be decoded to on chain. So, the user may signs an incorrect | ||
//! transaction leading to unexpecting behavior. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PolkadotDom ty, this is acutally a backport. If you like, please open a pr against master with your suggestions, ty!
This pull request has been mentioned on Polkadot Forum. There might be relevant details there: https://forum.polkadot.network/t/new-polkadot-ledger-app/8817/1 |
…#4619) This implements the `CheckMetadataHash` extension as described in [RFC78](https://polkadot-fellows.github.io/RFCs/approved/0078-merkleized-metadata.html). Besides the signed extension, the `substrate-wasm-builder` is extended to support generating the metadata-hash. Closes: paritytech#291 --------- Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io> Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com> Co-authored-by: Liam Aharon <liam.aharon@hotmail.com> Co-authored-by: Kian Paimani <5588131+kianenigma@users.noreply.github.com>
This implements the
CheckMetadataHash
extension as described in RFC78.Besides the signed extension, the
substrate-wasm-builder
is extended to support generating the metadata-hash.Closes: #291