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

Unnecessary crypto hashing #2

Open
tcrain opened this issue Apr 21, 2020 · 0 comments
Open

Unnecessary crypto hashing #2

tcrain opened this issue Apr 21, 2020 · 0 comments
Labels
performance Performance improvement

Comments

@tcrain
Copy link
Owner

tcrain commented Apr 21, 2020

There are points where crypto hashing is done even when not necessary, or done multiple times.
This especially is the case when not using signed messages.
Additionally, the hashes are sometimes used as keys in maps, where a shorter identifier should be used.

@tcrain tcrain added the performance Performance improvement label Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance improvement
Projects
None yet
Development

No branches or pull requests

1 participant