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

[Audit] Finish off the SMT audit #462

Closed
7 tasks
Olshansk opened this issue Apr 8, 2024 · 3 comments
Closed
7 tasks

[Audit] Finish off the SMT audit #462

Olshansk opened this issue Apr 8, 2024 · 3 comments
Assignees
Labels
Audit Anythings related to audits protocol General core protocol related changes research Related to open-ended research & investigation smt Sprase Merkle Tree Related

Comments

@Olshansk
Copy link
Member

Olshansk commented Apr 8, 2024

Objective

Address SMT audit concerns and lock-in MainNet SMT sha.

Origin Document

Goals

  • Address all audit concerns from the Thesis Defense team
  • Finalize and lock in a hash that can get Shannon through to MainNet

Deliverables

Non-goals / Non-deliverables

  • Prepare future work such as:
    • Optimizing proof sizes
    • Refactoring the code for readability & maintainbility

General deliverables

  • Comments: Add/update TODOs and comments alongside the source code so it is easier to follow.
  • Testing: Add new tests (unit and/or E2E) to the test suite.
  • Makefile: Add new targets to the Makefile to make the new functionality easier to use.
  • Documentation: Update architectural or development READMEs; use mermaid diagrams where appropriate.

Creator: @Olshansk

@Olshansk Olshansk added protocol General core protocol related changes research Related to open-ended research & investigation labels Apr 8, 2024
@Olshansk Olshansk added this to the Shannon Private TestNet milestone Apr 8, 2024
@Olshansk Olshansk self-assigned this Apr 8, 2024
@Olshansk Olshansk added smt Sprase Merkle Tree Related Audit Anythings related to audits labels Apr 8, 2024
@Olshansk
Copy link
Member Author

Olshansk commented Apr 9, 2024

First followup of questions: Reviewing Audit Report 80a867507a6e4ae0bfdea3667df00f1b.pdf

@Olshansk
Copy link
Member Author

Olshansk commented Apr 9, 2024

@bryanchriswhite @red-0ne @okdas For the curious ☝️

@Olshansk
Copy link
Member Author

The thesis defense team will follow up in about 3-4 weeks. Given that the scope of work here was to reply to the feedback and merge in the updates, I'm marking this as done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Audit Anythings related to audits protocol General core protocol related changes research Related to open-ended research & investigation smt Sprase Merkle Tree Related
Projects
Status: ✅ Done
Development

No branches or pull requests

1 participant