forked from ithacaxyz/account
-
Notifications
You must be signed in to change notification settings - Fork 1
Updated class diagram for IthacaAccount Merkle signature support #65
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededinvalidThis doesn't seem rightThis doesn't seem rightquestionFurther information is requestedFurther information is requestedwontfixThis will not be worked onThis will not be worked on
Description
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationduplicateThis issue or pull request already existsThis issue or pull request already existsenhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededinvalidThis doesn't seem rightThis doesn't seem rightquestionFurther information is requestedFurther information is requestedwontfixThis will not be worked onThis will not be worked on
Projects
StatusShow more project fields
Backlog
StatusShow more project fields
Todo
Reviewer's Guide
Adds Merkle tree–based signature wrapping/verification support to IthacaAccount, updates the wrapped-signature format to include a merkle flag, adjusts all test helpers and call sites accordingly, and introduces fuzz tests for Merkle signature validation while bumping the account contract version.
Updated class diagram for IthacaAccount Merkle signature support
classDiagram class IthacaAccount { +unwrapAndValidateSignature(bytes32 digest, bytes signature) bool isValid +unwrapAndValidateSignature(bytes32 digest, bytes signature) bytes32 keyHash -_verifyMerkleSig(bytes32 digest, bytes signature) bool isValid -_verifyMerkleSig(bytes32 digest, bytes signature) bytes32 keyHash +getNameAndVersion() string name +getNameAndVersion() string version "0.5.11" } class MerkleProofLib { <<library>> +verifyCalldata(bytes32[] proof, bytes32 root, bytes32 leaf) bool } class LibBytes { <<library>> +loadCalldata(bytes data, uint256 offset) bytes32 +truncatedCalldata(bytes data, uint256 newLength) bytes } class EfficientHashLib { <<library>> +sha2(bytes32 input) bytes32 } class ECDSA { <<library>> +recoverCalldata(bytes32 digest, bytes signature) address } class IIthacaAccount { <<interface>> } class EIP712 { } class GuardedExecutor { } IthacaAccount ..|> IIthacaAccount IthacaAccount --|> EIP712 IthacaAccount --|> GuardedExecutor IthacaAccount ..> MerkleProofLib : uses IthacaAccount ..> LibBytes : uses IthacaAccount ..> EfficientHashLib : uses IthacaAccount ..> ECDSA : usesFile-Level Changes
src/IthacaAccount.soltest/Base.t.soltest/Orchestrator.t.soltest/SimulateExecute.t.soltest/Account.t.solsrc/IthacaAccount.solPossibly linked issues
Tips and commands
Interacting with Sourcery
@sourcery-ai reviewon the pull request.issue from a review comment by replying to it. You can also reply to a
review comment with
@sourcery-ai issueto create an issue from it.@sourcery-aianywhere in the pullrequest title to generate a title at any time. You can also comment
@sourcery-ai titleon the pull request to (re-)generate the title at any time.@sourcery-ai summaryanywhere inthe pull request body to generate a PR summary at any time exactly where you
want it. You can also comment
@sourcery-ai summaryon the pull request to(re-)generate the summary at any time.
@sourcery-ai guideon the pullrequest to (re-)generate the reviewer's guide at any time.
@sourcery-ai resolveon thepull request to resolve all Sourcery comments. Useful if you've already
addressed all the comments and don't want to see them anymore.
@sourcery-ai dismisson the pullrequest to dismiss all existing Sourcery reviews. Especially useful if you
want to start fresh with a new review - don't forget to comment
@sourcery-ai reviewto trigger a new review!Customizing Your Experience
Access your dashboard to:
summary, the reviewer's guide, and others.
Getting Help
Originally posted by @sourcery-ai[bot] in #60 (comment)