Set default crypto_provider to SCrypt #396

merged 1 commit into from Mar 3, 2014


None yet
2 participants

Hermanverschooten commented Mar 1, 2014

The default crypto_provider is assumed to be SCrypt in the tests, but it was still set to Sha512.
This solves 2 of the failing password tests in #393.


binarylogic commented Mar 3, 2014

I'm actually not sure I want to change the default as it would break backwards compatibility. Anyone have any thoughts on this? I'd like to update the default as Sha512 is weak.


binarylogic commented Mar 3, 2014

I'm doing it and bumping the version, sha512 is outdated. Gotta move with the times.

binarylogic added a commit that referenced this pull request Mar 3, 2014

@binarylogic binarylogic merged commit 2bff3e5 into binarylogic:master Mar 3, 2014

@Hermanverschooten Hermanverschooten deleted the Hermanverschooten:fix_test_password branch Mar 5, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment