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

Reduce reliance on blake2 internals, move RequestCommon::*serialize* #2009

Merged
merged 4 commits into from
Jan 15, 2024

Conversation

hrxi
Copy link
Contributor

@hrxi hrxi commented Nov 23, 2023

No description provided.

@hrxi hrxi added the testnet-restart Change breaks the protocol and requires a testnet restart label Nov 23, 2023
@hrxi
Copy link
Contributor Author

hrxi commented Nov 23, 2023

Requires regenerating the ZKP keys.

Maybe we can get rid of the blake2-rfc dependency entirely once the blake2 crate supports blake2x: RustCrypto/hashes#83.

Simply add the personalization to the hashed data instead of using the
BLAKE2 personalization feature.
@jsdanielh jsdanielh merged commit 91701ed into albatross Jan 15, 2024
6 checks passed
@jsdanielh jsdanielh deleted the hrxi/minor_fixes41 branch January 15, 2024 20:42
@jsdanielh jsdanielh added this to the Nimiq PoS Mainnet milestone Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testnet-restart Change breaks the protocol and requires a testnet restart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants