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

Hash-based transaction validation #93

Open
pgebheim opened this issue Jul 12, 2022 · 2 comments
Open

Hash-based transaction validation #93

pgebheim opened this issue Jul 12, 2022 · 2 comments
Assignees
Labels

Comments

@pgebheim
Copy link

pgebheim commented Jul 12, 2022

Description

Currently the ledger app has a list of hardcoded transactions which are supported by the device. This is limiting because the device doesn't have much storage for each app, and it would be better to more efficiently verify transactions. The idea is the use a hash of the transaction as an identifier which then can be verified by an internal database.

There are a couple of options that are still TBD for how to actually store the set of valid hashes (e.g. as a list of hashes, or by construction a merkle trie and having each release contain the one root hash).

Definition of Done

TBD - @Wolog2021

@relatko
Copy link
Contributor

relatko commented Jul 13, 2022

Recent work included consolidation of data relevant to transaction implementation. The data consist more or less of

This approach is sufficient to store several hundreds of transactions. One of the tasks I am at the moment working on is move this data out of the app using standard Merkle tree technique (where the app will only have the root hash hardcoded). This will allow basically unlimited types of transactions the app can handle (practical limit will be probably the ability to provide the required data to the system I am developing).

@bluesign
Copy link

bluesign commented Jul 13, 2022

Still no dapp usage? Also why not signature but merkle hash?

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

No branches or pull requests

4 participants