private-attribution / ipa

A raw implementation of Interoperable Private Attribution
MIT License
40 stars 23 forks source link

Further simplifying the verifier code #1131

Closed benjaminsavage closed 2 months ago

benjaminsavage commented 2 months ago

We can further simplify the verifier code by just breaking down the verify_proof function into the two separate components.

I'm sorry the diff view is so unhelpful... I removed the ProofVerifier struct as it's no longer needed, and cargo fmt moved everything back one indentation... which is displayed so badly in the diff view.

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.51%. Comparing base (2f35e3e) to head (6ee608f).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1131 +/- ## ========================================== + Coverage 91.49% 91.51% +0.01% ========================================== Files 189 189 Lines 27316 27342 +26 ========================================== + Hits 24994 25023 +29 + Misses 2322 2319 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.