-
Notifications
You must be signed in to change notification settings - Fork 14
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
Choice of Hash Function #29
Conversation
No change, other than "sha1" option is passed into constructor. Test suite verifies current condition (prior to adding other hash algorithm options).
@trevor-vaughan export type HashAlgorithm = 'sha1' | 'sha256' | 'sha512'; I wonder, are there any other baseline/standard algorithms you'd like to see included?
Please let me know if you want any others, and I'll get them in and tested, and then bump this feature release over to NPM. Thanks! |
@philcockfield Ideally, you'd pull the available hashes out of the underlying installation and allow for any from that list. crypto = require('crypto')
hashes = crypto.getHashes() |
that's better @trevor-vaughan - cheers! |
Published as 2.4.0 |
Issue: #23