Skip to content

Comments

Use accounts-password and newer bcrypt#29

Open
adammoisa wants to merge 1 commit intookland:masterfrom
adammoisa:patch-1
Open

Use accounts-password and newer bcrypt#29
adammoisa wants to merge 1 commit intookland:masterfrom
adammoisa:patch-1

Conversation

@adammoisa
Copy link

Fixes node-gyp rebuild console error with bycrpt

@Exoool
Copy link

Exoool commented Aug 22, 2016

@adammoisa
With my local project, I had to use the version of 0.9.1 instead of 0.8.x.
Later I tried with get rid of the version number completely - and it works just fine by going defaut to v0.9.1.

Any idea about the drawbacks without specifying the version number?

@adammoisa
Copy link
Author

@Exoool I unfortunately don't know the details of the bcrypt version differences, I guess do whatever works for you, a later version is (usually considered) better. If you come across a unique bug with versioning please let the community know!

@Exoool
Copy link

Exoool commented Aug 23, 2016

@adammoisa Sure, it was the same bug whatsoever - just wasn't sure if it's perfectly ok to remove the version number. But now as it works in reality so I guess it's just fine!

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.

3 participants