Skip to content

Crypto constants#115

Draft
tarakby wants to merge 3 commits intoonflow:developfrom
tarakby:crypto-constants
Draft

Crypto constants#115
tarakby wants to merge 3 commits intoonflow:developfrom
tarakby:crypto-constants

Conversation

@tarakby
Copy link
Contributor

@tarakby tarakby commented Feb 14, 2025

  • declare new cryptographic constants related to sha3 and the supported elliptic curves.
  • use the cryptographic constants to replace magic numbers

The purpose is:

  • clarify the intention of using magic numbers (especially when the same constant 32 is used multiple times, but it has a different meaning depending on the context)
  • make the code base more robust if more hashing algorithms or elliptic curves are used in the future

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.

1 participant

Comments