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

Support for all available hash functions to compute message digests #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vitalyisaev2
Copy link

@vitalyisaev2 vitalyisaev2 commented Jul 31, 2019

Please consider this PR that provides an opportunity to utilize any hash function impemented in OpenSSL:

$ openssl list --digest-commands
blake2b512        blake2s256        gost              md2               
md4               md5               rmd160            sha1              
sha224            sha256            sha3-224          sha3-256          
sha3-384          sha3-512          sha384            sha512            
sha512-224        sha512-256        shake128          shake256          
sm3 

New generic entity DigestComputer is introduced. Existing types like SHA256Hash or MD5Hash now embed this structure to preserve backward compatibility, but prevent code duplication. All the other digests (like BLAKE2 etc.) can be calculated with the direct use of DigestComputer

@vitalyisaev2 vitalyisaev2 changed the title Support all available hash functions to compute message digests Support for all available hash functions to compute message digests Jul 31, 2019
@vitalyisaev2 vitalyisaev2 changed the title Support for all available hash functions to compute message digests WIP: Support for all available hash functions to compute message digests Aug 6, 2019
@vitalyisaev2 vitalyisaev2 changed the title WIP: Support for all available hash functions to compute message digests Support for all available hash functions to compute message digests Aug 8, 2019
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