private-attribution / ipa

A raw implementation of Interoperable Private Attribution
MIT License
42 stars 25 forks source link

Hybrid oprf pt2 #1413

Closed eriktaubeneck closed 3 weeks ago

eriktaubeneck commented 3 weeks ago

This is stacked on top of #1412, so to look at the just the diff, make sure to ignore the d1199d1 commit in the diff viewer.

This updates the OPRF function to work with hybrid reports, as well as negotiating a key exchange across shards. Note that that will be replaced by the functionality in #1410 once merged.

It also adds a direct test for this function, which was lacking before.

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 89.47368% with 18 lines in your changes missing coverage. Please review.

Project coverage is 93.27%. Comparing base (f587a37) to head (4e00be9). Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
ipa-core/src/query/runner/hybrid.rs 70.00% 18 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1413 +/- ## ========================================== + Coverage 93.17% 93.27% +0.10% ========================================== Files 225 225 Lines 38569 38659 +90 ========================================== + Hits 35936 36060 +124 + Misses 2633 2599 -34 ```

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