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

Multiple functions with the same names #385

Open
jyasskin opened this issue Jul 13, 2023 · 1 comment
Open

Multiple functions with the same names #385

jyasskin opened this issue Jul 13, 2023 · 1 comment

Comments

@jyasskin
Copy link

The current draft has def BlindEvaluate in https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-oprf-protocol, https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-voprf-protocol, and https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-poprf-protocol. def Blind is in https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-oprf-protocol and https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-poprf-protocol. def Evaluate is in https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-oprf-protocol and https://www.ietf.org/archive/id/draft-irtf-cfrg-voprf-21.html#name-poprf-protocol.

When writing a specification that depends on one of this draft's protocols (as I'm currently reviewing in WICG/trust-token-api#257), the specification needs to call the appropriate function. While it's possible to link to a specific section or name the active protocol so that a careful reader will implement the right function, it would be better if we didn't have to be so careful.

Would it be possible to give every distinct function in this spec a distinct name?

@armfazh
Copy link
Collaborator

armfazh commented Jul 13, 2023

The name of the primitives are generic, similar to Encrypt/Decrypt or Sign/Verify.
All the variants follow this abstraction, so for the sake of clarity, specifications should announce which variant (OPRF/VOPRF/POPRF) is used. Additionally, the draft provides test vectors and reference implementations so people can verify their implementation as an additional measure.

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

No branches or pull requests

2 participants