Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align XofTurboShake128 with VDAF-13 #1165

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

cjpatton
Copy link
Collaborator

Partially addresses #1122.

  • Bump seed size for XofTurboShake128 to 32 bytes
  • Allow variable length seeds for compatibility with IDPF.
  • Make the dst length prefix 2 bytes

@cjpatton cjpatton force-pushed the cjpatton/1122-xof-turboshake-seed-size branch from c14dd95 to 6a05f3a Compare December 16, 2024 20:13
tests/discrete_gauss.rs Outdated Show resolved Hide resolved
@cjpatton cjpatton force-pushed the cjpatton/1122-xof-turboshake-seed-size branch from 6a05f3a to f62005d Compare December 16, 2024 20:18
@cjpatton cjpatton marked this pull request as ready for review December 16, 2024 20:25
@cjpatton cjpatton requested a review from a team as a code owner December 16, 2024 20:25
@cjpatton cjpatton force-pushed the cjpatton/1122-xof-turboshake-seed-size branch from d506959 to 0e85810 Compare December 16, 2024 22:29
src/vdaf/xof.rs Outdated Show resolved Hide resolved
src/vdaf/poplar1.rs Outdated Show resolved Hide resolved
src/flp/szk.rs Outdated Show resolved Hide resolved
* Bump seed size for XofTurboShake128 to 32 bytes
* Allow variable length seeds for compatibility with IDPF.
* Make the dst length prefix 2 bytes
@cjpatton cjpatton force-pushed the cjpatton/1122-xof-turboshake-seed-size branch from f4bf473 to 50a1ae3 Compare December 16, 2024 23:52
@divergentdave divergentdave merged commit 3ad1230 into main Dec 17, 2024
6 checks passed
@divergentdave divergentdave deleted the cjpatton/1122-xof-turboshake-seed-size branch December 17, 2024 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants