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.

Sign up for GitHub

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

Jump to bottom

fix(encode) calculate appropriate encoded_len #21

Merged

thibaultcha merged 6 commits into master from feat/variable-len

Nov 29, 2016

Owner

thibaultcha commented Nov 29, 2016

No description provided.

coveralls commented Nov 29, 2016 •

edited

Coverage increased (+0.007%) to 99.138% when pulling 0ae45f0 on feat/variable-len into c6953ae on master.

thibaultcha added 2 commits

November 28, 2016 23:53


          chore(makefile) fix linker

de9dca6


          fix(encode) calculate appropriate encoded_len

2919d31

Use 'argon2_encodedlen()' to determine the appropriate length of the
encoded hash. 108 (old constant value) turned out to not always be
desired/big enough, causing failures to encode especially with long
salts.

See thibaultcha/lua-argon2-ffi#8

thibaultcha force-pushed the feat/variable-len branch from 0ae45f0 to 2919d31 Compare

November 29, 2016 07:53

coveralls commented Nov 29, 2016 •

edited

Coverage increased (+0.007%) to 99.138% when pulling 2919d31 on feat/variable-len into c6953ae on master.

2 similar comments

coveralls commented Nov 29, 2016

Coverage increased (+0.007%) to 99.138% when pulling 2919d31 on feat/variable-len into c6953ae on master.

coveralls commented Nov 29, 2016

Coverage increased (+0.007%) to 99.138% when pulling 2919d31 on feat/variable-len into c6953ae on master.

thibaultcha added 2 commits

November 29, 2016 00:28


          style(argon2) drop clang-format, manual format for now

94fa58f


          fix(argon2) avoid getting config upvalue in largon2_verify and better

398777f

"too many arguments" error messages

coveralls commented Nov 29, 2016

Coverage decreased (-0.8%) to 98.291% when pulling 398777f on feat/variable-len into c6953ae on master.

4 similar comments

coveralls commented Nov 29, 2016

Coverage decreased (-0.8%) to 98.291% when pulling 398777f on feat/variable-len into c6953ae on master.

coveralls commented Nov 29, 2016

Coverage decreased (-0.8%) to 98.291% when pulling 398777f on feat/variable-len into c6953ae on master.

coveralls commented Nov 29, 2016

Coverage decreased (-0.8%) to 98.291% when pulling 398777f on feat/variable-len into c6953ae on master.

coveralls commented Nov 29, 2016

Coverage decreased (-0.8%) to 98.291% when pulling 398777f on feat/variable-len into c6953ae on master.


          chore: drop support notices for Argon2 < 20161029

b790c27

coveralls commented Nov 29, 2016 •

edited

Coverage decreased (-0.8%) to 98.291% when pulling b790c27 on feat/variable-len into c6953ae on master.


          style(argon2) remove extraneous return statement

09fbc7a

coveralls commented Nov 29, 2016 •

edited

Coverage increased (+0.01%) to 99.145% when pulling 09fbc7a on feat/variable-len into c6953ae on master.

thibaultcha merged commit 9b0ea1d into master

thibaultcha deleted the feat/variable-len branch

November 29, 2016 19:28

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