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 transactions verification in client library #1468

Closed
11 tasks done
Tracked by #1313
jpraynaud opened this issue Jan 29, 2024 · 0 comments · Fixed by #1489
Closed
11 tasks done
Tracked by #1313

Implement transactions verification in client library #1468

jpraynaud opened this issue Jan 29, 2024 · 0 comments · Fixed by #1489
Assignees
Labels
feature 🚀 New implemented feature

Comments

@jpraynaud
Copy link
Member

jpraynaud commented Jan 29, 2024

Why

The client library needs to provide a way to generate, verify and certify proofs for a list of transactions.

What

Implement new API function for verifying a list of proofs generated by the aggregator in the client library.

How

  • Implement transaction verification function in library:
    • Call aggregator to generate a list of proofs for the transaction list
    • Return the list of certified transactions (and list of not certified?)
    • Verify the proofs for all the certified transactions
    • Retrieve the latest artifact (and its certificate hash)
    • Verify that the Merkle root of the Merkle proofs is signed by the certificate
    • Verify that the certificate chain is valid
  • Implement a function that lists artifacts or get latest produced artifact?
  • Update Mithril events feedback mechanism
  • Update documentation
  • Add feature flag unstable?

Later

  • Add an example in the repository (later, will be done in another PR)
@jpraynaud jpraynaud added the feature 🚀 New implemented feature label Jan 29, 2024
@jpraynaud jpraynaud changed the title Light Wallet: Implement transactions verification in client library Implement transactions verification in client library Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 🚀 New implemented feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants