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.

Sign up for GitHub

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

Jump to bottom

Update Test Vectors to VOPRF draft version 17 #382

Closed
daxpedda opened this issue Feb 1, 2023 · 2 comments
Closed

Update Test Vectors to VOPRF draft version 17 #382

daxpedda opened this issue Feb 1, 2023 · 2 comments
Labels
next-version Issue to fix in next version

Comments

@daxpedda
Copy link
Contributor

daxpedda commented Feb 1, 2023 •
edited
Loading

Since VOPRF draft version 17 the Ciphersuite IDs have changed to being strings instead of a simple 2 byte integers.
Additionally the CreateContextString function changed the first string to OPRFV1- and added a - before the Ciphersuite ID.

I have also noticed that the editor's copy actually links to the VOPRF draft version 17, even though the test vectors don't seem to have been updated.
@daxpedda daxpedda mentioned this issue Feb 1, 2023
Merged
@bytemare
Copy link
Collaborator

bytemare commented Feb 6, 2023

Hi @daxpedda 👋
cfrg/draft-irtf-cfrg-voprf#347 seems to be modifying the poc API. I'll wait for this to be merged to update our poc (even though it has no impact on the vectors).

@bytemare bytemare added the next-version Issue to fix in next version label Feb 6, 2023
@chris-wood
Copy link
Collaborator

This should be done now. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
next-version Issue to fix in next version
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@daxpedda @chris-wood @bytemare