bcrypt is an optional dependency

Author: sokra

lib/ldapauth.js

@@ -13,7 +13,6 @@
  */
 
 var assert = require('assert');
-var bcrypt = require('bcrypt');
 var ldap = require('ldapjs');
 var debug = console.warn;
 var format = require('util').format;
@@ -84,7 +83,10 @@ function LdapAuth(opts) {
   this._adminBound = false;
   this._userClient = ldap.createClient(clientOpts);
 
-  this._salt = bcrypt.genSaltSync();
+  if (opts.cache) {
+    var bcrypt = require('bcrypt');
+    this._salt = bcrypt.genSaltSync();
+  }
 }
 
 
@@ -190,6 +192,7 @@ LdapAuth.prototype.authenticate = function (username, password, callback) {
   if (self.opts.cache) {
     // Check cache. 'cached' is `{password: <hashed-password>, user: <user>}`.
     var cached = self.userCache.get(username);
+    var bcrypt = require('bcrypt');
     if (cached && bcrypt.compareSync(password, cached.password)) {
       return callback(null, cached.user)
     }
@@ -208,6 +211,7 @@ LdapAuth.prototype.authenticate = function (username, password, callback) {
         return callback(err);
       }
       if (self.opts.cache) {
+        var bcrypt = require('bcrypt');
         bcrypt.hash(password, self._salt, function (err, hash) {
           self.userCache.set(username, {password: hash, user: user});
           return callback(null, user);

package.json

@@ -23,7 +23,9 @@
   ],
   "dependencies": {
     "ldapjs": "0.6.3",
-    "bcrypt": "0.7.5",
     "lru-cache": "2.0.4"
+  },
+  "optionalDependencies": {
+    "bcrypt": "0.7.5"
   }
 }