Skip to content

Conversation

@daxpedda
Copy link
Contributor

@daxpedda daxpedda commented Feb 1, 2023

After #1175 got merged I attempted to go ahead and update voprf and opaque-ke, but quickly hit the same problem that required #876.

The change to the OPRF ID now being a string instead of a byte made it impossible to generate a fixed size array to hold the DST. So I decided to apply the same solution I did in #876.

There are only two changes:

  • All hash2curve APIs now take &[&[u8]] instead of &[u8] as the dst parameter.
  • ExpandMsgXmd now requires Default + FixedOutput + HashMarker instead of Digest.

@daxpedda daxpedda force-pushed the hash2curve-multi-dst branch from 3744148 to 9efa183 Compare February 1, 2023 13:03
@tarcieri tarcieri merged commit 771f5df into RustCrypto:master Feb 1, 2023
@tarcieri
Copy link
Member

tarcieri commented Feb 1, 2023

@daxpedda if you can update https://github.com/RustCrypto/elliptic-curves that'd be great. It's pulling in elliptic-curve from git again

@daxpedda
Copy link
Contributor Author

daxpedda commented Feb 1, 2023

On it.

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