Closed eriktaubeneck closed 1 month ago
Attention: Patch coverage is 89.67972%
with 29 lines
in your changes missing coverage. Please review.
Project coverage is 93.46%. Comparing base (
4307265
) to head (589637d
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This makes decryption and uniqueness verification work in a sharded context.
Because we cannot reshard dynamically sized structs (e.g., an EncryptedHybridReport), this takes the approach of:
Now, this is not idea because we have to complete all decryption before we can do the uniqueness check. I attempted to try and get this working with streaming, but ran into a bunch of issues. Still happy to keep trying that approach, but wanted to get this up to get some feedback.