* changing options.ldap check to instance of instead of is function

bhoriuchi · Aug 30, 2016 · 04cb4e7 · 04cb4e7
1 parent 8962a18
commit 04cb4e7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/index.js b/index.js
@@ -53,7 +53,7 @@ function Strategy(options, verify) {
     this._passwordField = options.passwordField || DEFAULT_PASSWORD_FIELD;
   }
 
-  this._ad = typeof options.ldap === 'function' ? options.ldap : new ActiveDirectory(options.ldap);
+  this._ad = options.ldap instanceof ActiveDirectory ? options.ldap : new ActiveDirectory(options.ldap);
 }
 
 util.inherits(Strategy, passport.Strategy);

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "passport-activedirectory",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Active Directory strategy for passport.js",
   "license": "MIT",
   "main": "index.js",
@@ -19,9 +19,9 @@
     "passport": "^0.3.0"
   },
   "devDependencies": {
-    "rollup": "^0.31.2",
-    "rollup-plugin-babel": "^2.5.1",
-    "babel-preset-es2015-rollup": "^1.1.1"
+    "rollup": "^0.34.10",
+    "rollup-plugin-babel": "^2.6.1",
+    "babel-preset-es2015-rollup": "^1.2.0"
   },
   "bugs": {
     "url": "https://github.com/bhoriuchi/passport-activedirectory/issues"

diff --git a/src/strategy.js b/src/strategy.js
@@ -59,7 +59,7 @@ function Strategy (options, verify) {
     this._passwordField = options.passwordField || DEFAULT_PASSWORD_FIELD;
   }
 
-  this._ad = (typeof options.ldap === 'function') ? options.ldap : new ActiveDirectory(options.ldap)
+  this._ad = (options.ldap instanceof ActiveDirectory) ? options.ldap : new ActiveDirectory(options.ldap)
 }
 
 util.inherits(Strategy, passport.Strategy)