Npm bcrypt and meteor npm bcrypt madness

autoschematic · 2016-08-30T15:57:41.617Z

Somewhere between version 1.3 and 1.4 of meteor I had to install a specific version of bcrypt because of some accounts-ui issue - I’ve since lost track of that github issue so I’m having trouble going back over what I’ve changed.

Now I keep getting this error:

W20160830-11:50:40.866(-4)? (STDERR) Note: you are using a pure-JavaScript implementation of bcrypt.
W20160830-11:50:40.868(-4)? (STDERR) While this implementation will work correctly, it is known to be
W20160830-11:50:40.868(-4)? (STDERR) approximately three times slower than the native implementation.
W20160830-11:50:40.868(-4)? (STDERR) In order to use the native implementation instead, run
W20160830-11:50:40.869(-4)? (STDERR)          
W20160830-11:50:40.869(-4)? (STDERR)   meteor npm install --save bcrypt
W20160830-11:50:40.869(-4)? (STDERR)          
W20160830-11:50:40.870(-4)? (STDERR) in the root directory of your application.

I’ve run meteor npm install --save bcrypt multiple times and the error will go away, but just until I do something with npm install [somepackage] --save - every time I do, this error comes back.

What’s the deal, anyone else experiencing this?

Does anyone know how to stop this madness?

mcb · 2016-08-30T20:02:09.946Z

I have the same issue. Don’t know of a fix

lucfranken · 2016-08-30T20:35:51.546Z

Did you try installing that bcrypt version globally?

autoschematic · 2016-08-31T00:04:22.008Z

How would I install the “Meteor” version globally? Meteor complains about the pure javascript version being slow and wants to install their super neat speedy one…

lucfranken · 2016-08-31T08:31:53.379Z

Sorry, with expected I mean you install it just with npm install -g. But I reviewed some installs here and that shouldn’t be needed.

Do you have this in your package.json?:

  "dependencies": {
    "bcrypt": "^0.8.7"
  },

henribeck · 2016-08-31T11:41:40.776Z

Do you still have the bcrypt package in your .meteor/packages file?

lucfranken · 2016-08-31T11:47:51.677Z

henribeck:

Do you still have the bcrypt package in your .meteor/packages file?

Yes, no idea whether it’s needed, I have in package and versions file:

npm-bcrypt@0.9.1

Will put that on our todo to see whether it’s a duplicate or actually necessary.

henribeck · 2016-08-31T11:48:31.430Z

Try removing the bcrypt package from the packages file

lucfranken · 2016-08-31T11:49:11.991Z

henribeck:

Try removing the bcrypt package from the packages file

my setup works, I am not @autoschematic

sessionwire · 2016-08-31T17:28:07.140Z

I am experiencing the same issue.

autoschematic · 2016-08-31T17:38:19.170Z

Hi everyone again,

So, to answer your question @henribeck

henribeck:

Try removing the bcrypt package from the packages file

I’ve tried pretty much everything and it’s still giving me this error on and off.

I’ve removed bcrypt npm remove bcrypt --save (and looked at the packages.json file to double check that "bcrypt": "^0.8.7" is not there) and then ran the meteor npm install --save bcrypt - and then it places this:

"dependencies": {
  "bcrypt": "^0.8.7",
}

in my packages.json

I really wish I could track down what that github issue was talking about concerning this - it was a fix related to the accounts-ui or account-core or something… I ran that fix, and It was related to installing bcrypt. I’ve checked to see if bcrypt is installed globally, and it doesn’t seam to be.

Any thoughts?

Edit: I mean, obviously this isn’t a breaking thing - it’s just annoying to see the pink error

henribeck · 2016-08-31T18:03:08.556Z

No, i mean remove bcrypt .meteor/packages file, not from you’re package.json file

gemmi · 2016-09-06T10:46:52.295Z

I fixed this error with:
ls node_modules | grep bcrypt
rm -rf node_modules/bcrypt-pbkdf
meteor npm install --save bcrypt

autoschematic · 2016-09-07T15:58:26.391Z

@gemmi & @henribeck, thanks so much - If it comes up again, I’ll try what you did there Gemmi, along with making sure bcrypt is completely uninstalled from both package files and report back.

tomsp · 2016-09-12T11:07:59.431Z

So accounts-password depends on npm-bcrypt which I think is the culprit here. It adds npm-bcrypt@0.9.1 to my meteor/versions files.

hoebestaathet · 2016-09-14T07:41:14.016Z

I’ve used meteor npm rebuild after that I didn’t see the notice again.
Mention here: https://guide.meteor.com/1.4-migration.html#binary-packages-require-build-toolchain

satya · 2016-09-14T10:18:43.297Z

meteor npm rebuild not working for me…

lednhatkhanh · 2016-09-14T10:36:47.153Z

See my answer here:

How to fix the bcrypt error in windows 7? help

I updated my meteor project to 1.4.1.1 , it says that you are using pure JS implementation of bcrypt. [image] then i use meteor npm install --save bcrypt it gives me a lot of errors and i don`t know how to fix it … [image]

aadams · 2016-09-14T15:25:24.688Z

Did you get this working?

mikkelking · 2016-09-22T01:15:43.259Z

This is driving me nuts… I tried the above remedies, none of which fixed it.

Now it complains every time I run meteor, even after issuing the command to install bcrypt

npm rebuild didn’t help. Trying to remove node-bcrypt didn’t work, Meteor just put it back in there.

Can someone please provide a fix for this?

In fact, while I’m on my soapbox, Meteor will tell me that an npm module is missing and may cause problems. Why can’t it just install the package, or at least prompt me to do so? A common problem among the dev team is caused by this, after doing a “git pull” an “npm install” is necessary to install any newly introduced packages. This is a time waster that (I think) could be fixed quite easily. After all, if the .meteor/packages file has a new entry, Meteor will fetch that and install it. Can’t it do the same with npm modules?