Closed h5law closed 6 months ago
Attention: Patch coverage is 76.00000%
with 12 lines
in your changes are missing coverage. Please review.
Project coverage is 82.87%. Comparing base (
8682379
) to head (46dc5c5
).
Files | Patch % | Lines |
---|---|---|
proofs.go | 57.14% | 12 Missing :warning: |
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary
Human Summary
This PR addresses Issues A and B by providing extra documentation related to the usage of hashing algorithms and the writability of the key-value store. As these are related to the usage of the SMT documentation guidance on best practices for the security of the trie is needed.
Suggestions 1 and 3 have been addressed with the consolidation of the
ClosestProof
verification method's API and the way in whichExtentionNodes
are created with regards to the usage of shared pointers.AI Summary
reviewpad:summary
Issue
Fixes N/A
Audit Report: Audit Report.pdf
Type of change
Please mark the relevant option(s):
Testing
make test_all
make benchmark_{all | suite name}
Required Checklist
godoc
format comments see: tip.golang.org/doc/comment)If Applicable Checklist