JackAdams / meteor-accounts-ldap

Authentication against an LDAP server in Meteor
MIT License
21 stars 12 forks source link

Loads of errors if the Meteor package 'accounts-password' is not already installed #14

Open Suppen opened 7 years ago

Suppen commented 7 years ago

Specifically, not having accounts-password installed results in errors because the installation process tries to install bcrypt 0.7.8, which does not compile for node 4.

Installing accounts-password manually for some reason installs the correct bcrypt version, and installing babrahams:accounts-ldap later works

Not sure if this is possible to fix in this project, but at least this issue will notify others

JackAdams commented 7 years ago

Thanks for raising this issue, getting to the bottom of it and providing a solution. I'm pretty ignorant about how all the dependencies work when npm and Meteor packages are mixed up together. I'll leave this issue open and may not fix it if it proves too difficult.

Suppen commented 7 years ago

Could be worth mentioning in the readme?